-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Require latest version of WPCS always #37
Comments
https://github.com/jhabdas/wp-enforcer/blob/develop/CHANGELOG.md#042---2017-01-04
Where are those changes? |
@stevegrunwell could you let me know what versioning tool and changelog generator you are using to manage this project since they're not installed as a dev dependency? I may need to fork and want to be able to merge back together. |
That particular change was in a521897, where the version number was bumped but, as you pointed out, the version number is still getting locked-in.
The versioning of the plugin is manual: each feature branch includes a relevant change to the CHANGELOG.md file (under the "[Unreleased]" header). When a new release is being prepared, I branch This approach ensures people can easily see features/fixes in |
Please see #38 (comment) for my suggestion on how to enable future upgrades. Note that before dependencies go to |
hi @stevegrunwell @JHabdas I'm literally in the process of introducing this into our workflow, and I need to be able use some of the custom sniff rules that were introduced in WPCS 0.11.0 and 0.12.0. How can I go about this independently, before the version bump is made in this repo? Is it just a case of specifying a more recent version in Any guidance appreciated. Fantastic tool! |
Hey @davetgreen, I just tagged version 0.5.0, which follows @JHabdas' recommendation to use |
@stevegrunwell you sir are an absolute legend Much obliged /tiphat |
Currently
composer.json
has a hard dependency on WPCS, and is using0.10.0
. Because they're not at a 1.0 and the current rule is^
this change will not be automatic for minor updates (only one a major lands will that be true). As a result, we need to update to the new standard manually.The text was updated successfully, but these errors were encountered: