4.x #487
Merged
4.x #487
Conversation
This file contains 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
so for a single group a single process is enough Signed-off-by: Konrad Weihmann <[email protected]>
which holds all the configuration defined by arguments, rule files and config files. It makes all the global variables obsolete, which aren't shared between processes, so all code from rule_file and color have been removed Signed-off-by: Konrad Weihmann <[email protected]>
as this is not a runtime requirement Signed-off-by: Konrad Weihmann <[email protected]>
- refactor all rules - add typing hints to all code - update the example custom rule - remove `--constantfile` option, as this is not supported by the parser anymore (long deprecated option) - add new tests for missing bits of coverage Signed-off-by: Konrad Weihmann <[email protected]>
- add an option called --release so one can specify a release of YP to run against - default the latest - add a table that enables the new override syntax from kirkstone on and removes AUTHOR var from nanbield on - new tweaks call auto applies those tweaks from the first version up to the one specified by CLI - new tweaks class also add a new args var called _release_range that holds all the release names which are covered by the current config - also add a way to alter constantmods in args to use dictionaries as inserted by tweaks class - remove suggested var AUTHOR from tests, as in the default latest release this isn't applicable anymore Closes #481 Signed-off-by: Konrad Weihmann <[email protected]>
like loading up/from a certain release, by adding the new function `check_release_range` to Rule class. That one can also be overridden to enable even more release specific tweaks. Adds also tests and documentation of this new feature Signed-off-by: Konrad Weihmann <[email protected]>
Signed-off-by: Konrad Weihmann <[email protected]>
by collecting the found and applicable suppressions and comparing them against what was set in the check code. This comes with a special rule that is only used by main. Closes #362 Signed-off-by: Konrad Weihmann <[email protected]>
when calling finding from the rule code. So we could have different severities from the same rule code Signed-off-by: Konrad Weihmann <[email protected]>
for variables that are not known from misspell checks, but do not have a close match. Make it an info level finding, so people won't be bothered too much with it Signed-off-by: Konrad Weihmann <[email protected]>
with mickledore and later releases from variables/known as pointed out by the mickledore release notes Signed-off-by: Konrad Weihmann <[email protected]>
but only for nanbield and later Signed-off-by: Konrad Weihmann <[email protected]>
as moaned about by very latest flake8-bugbear Signed-off-by: Konrad Weihmann <[email protected]>
to check on addpylib being used correctly Signed-off-by: Konrad Weihmann <[email protected]>
Signed-off-by: Konrad Weihmann <[email protected]>
Signed-off-by: Konrad Weihmann <[email protected]>
of inherit and inherit_defer statements Closes #477 Signed-off-by: Konrad Weihmann <[email protected]>
as scarthgap is still a moving target Signed-off-by: Konrad Weihmann <[email protected]>
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.
Pull request checklist
Bugfix
New feature