Conversation
Rodrigo did today's release while in a call with me. Here are some small updates to the release checklist (and the README) based on that experience.
Release checklist: minor updates
Bumps the action-runners group with 1 update: [actions/checkout](https://github.com/actions/checkout). Updates `actions/checkout` from 7.0.0 to 7.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@9c091bb...3d3c42e) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: action-runners ... Signed-off-by: dependabot[bot] <support@github.com>
…ion-runners-7a5a078ad4 GH Actions: Bump actions/checkout from 7.0.0 to 7.0.1 in the action-runners group
* Add `SECURITY.md` file The other day, Rodrigo and me noticed that WPCS does not have a published security policy (via a `SECURITY.md` file). This commit intends to add such a file, which should help inform security researchers how to disclose any findings they may have. The file is placed in the `.github` directory. This will allow for it to be recognized correctly by GitHub, while not cluttering up the project root directory. Ref: https://docs.github.com/en/code-security/how-tos/report-and-fix-vulnerabilities/configure-vulnerability-reporting/add-security-policy --------- Co-authored-by: jrfnl <jrfnl@users.noreply.github.com> Co-authored-by: Denis Žoljom <dingo-d@users.noreply.github.com>
PHPCSUtils has just released version 1.2.3, which includes a security fix which is relevant for two sniffs which are included in PHPCSExtra, which has released version 1.5.1 in response. With that in mind, I'm bumping the minimum supported PHPCSUtils and PHPCSExtra versions to encourage users of to upgrade the underlying dependencies. Why does a version bump help ? Well, it only helps when a new version of WordPressCS is released, which is what I intend to do next. Once that's done, it helps as Composer will now show WordPressCS as a direct dependency which is outdated, while PHPCSUtils/PHPCSExtra would only show as an indirect dependency which is outdated, which doesn't have the same urgency for people to update. Refs: * https://github.com/PHPCSStandards/PHPCSUtils/releases/tag/1.2.3 * https://github.com/PHPCSStandards/PHPCSExtra/releases/tag/1.5.1
Composer: update minimum version PHPCSUtils + PHPCSExtra
This PR is a fix for CVE-2026-45293 / [GHSA-3pwp-g2mj-5p3v](GHSA-3pwp-g2mj-5p3v). Replace the eval()-based logic with explicit token-level checks for a small set of literal falsy values: - Boolean false. - An integer or float equal to zero. - A text string with the content `'0'` or `''` (single or double-quoted, heredoc, or nowdoc). - An empty array. Other forms that the previous implementation recognised via eval() are no longer detected, such as a value wrapped in a type cast (e.g. `(int) 0`). Supporting them doesn't justify the risks of using eval(). This commit also clarifies test case comments as the previous version was inaccurate. "0, false or NULL" are not the only values that are not allowed. Also, NULL and missing $ver parameter generate a warning instead of an error.
…ms-fix-is-falsy WP/EnqueuedResourceParameters: remove eval() from is_falsy()
The release will be tagged momentarily.
Changelog for the release of WordPressCS 3.4.1
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR for tracking changes for the 3.4.1 release. Target release date: NOW.
Release checklist
General
composer.json- PR Composer: update minimum version PHPCSUtils + PHPCSExtra #2770MinimumWPVersionTrait::$default_minimum_wp_versionand if so, action it - PR #xxxminimum_wp_versionandtestVersionproperties inphpcs.xml.dist.sampleneed updating and if so, action it - PR #xxxtestVersionmentioned in the README needs updating and if so, action it - PR #xxx✏️ Rule of thumb: stay in line with the "support three versions of WP below the current version" guideline, as also applied for the
minimum_wp_version.✏️ Make sure the "last updated" annotation in the docblocks for these lists has also been updated!
List based sniffs:
WordPress.WP.ClassNameCase- PR #xxxWordPress.WP.DeprecatedClasses- PR #xxxWordPress.WP.DeprecatedFunctions- PR #xxxWordPress.WP.DeprecatedParameters- PR #xxxWordPress.WP.DeprecatedParameterValues- PR #xxx$allowed_core_constantsinWordPress.NamingConventions.PrefixAllGlobals- PR #xxx$pluggable_functionsinWordPress.NamingConventions.PrefixAllGlobals- PR #xxx$pluggable_classesinWordPress.NamingConventions.PrefixAllGlobals- PR #xxx$target_functionsinWordPress.Security.PluginMenuSlug- PR #xxx$reserved_namesinWordPress.NamingConventions.ValidPostTypeSlug- PR #xxx$wp_time_constantsinWordPress.WP.CronInterval- PR #xxx$known_test_classesinIsUnitTestTrait- PR #xxxRelease prep
✏️ Remember to add a release link at the bottom!
README(if applicable) - PR #xxxRelease
main(careful, GH defaults todevelop!) & copy & paste the changelog to it.From within the GH interface: Code tab -> Releases -> "Draft a new release" button at the top of the page.
✏️ Check if anything from the link collection at the bottom of the changelog needs to be copied in!
developto be equal tomain(git checkout develop && git pull upstream/develop && git merge main).📝 Branch protection may need to get a temporary exception to allow for pushing the
developbranch. Please remember to remove the exception once the push has gone through!After release
Publicize
✏️ No need to post in the #core-coding-standard channel as that gets an automated release notification anyway.
Example post: [AMPLIFY]: WordPress Coding Standards 3.3.0 was released today marketing#746