-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update rule selection to respect preview mode #7195
Merged
Merged
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
9b4aee1
Rename a bunch of "nursery" references to "preview"
zanieb 8fce454
Update ALL rule selector to include all rules then deselect preview r…
zanieb c527f69
Clarify `include_preview_rules` conditional
zanieb a954fd6
Fix conditional
zanieb 2e44d66
WIP: Rename rule selector from nursery to preview
zanieb 51bce02
Enable selection of preview rules
charliermarsh 1689190
Make preview part of the selector API
charliermarsh 2c830e2
Fix linter group comment
zanieb f111189
Add comment to `PerFileIgnore`
zanieb e6d491a
Restore nursery rule group for backwards compatible selection
zanieb 97b80c0
Restore nursery selector
zanieb 2b0dfd6
Fix macro generated `RuleSelector::from(code)` support for `RuleSelec…
zanieb 359012a
Fix pyproject test
zanieb b60ce8b
Remove `prefix_to_selector`
zanieb 950bd95
Update FAQ
zanieb ae80063
Fix benchmark rule selection
zanieb 5ae0075
Clippy
zanieb a5200ed
Fix typo
zanieb 83d888f
Add support for NURSERY selector (backwards compat)
zanieb 45ceb6a
Disable preview mode during benchmarks
zanieb cb47f02
Separate test cases for selection
zanieb 2d8f961
Add preview flag to test utility
zanieb 550f787
Add tests for preview / nursery
zanieb c581006
Add test for select of linter; fix names of existing tests
zanieb f628e95
Add test cases for selection of NURSERY and PREVIEW groups
zanieb c092d62
Fix typo
zanieb 32b34fa
Add unreachable code to test cases
zanieb fbd449a
Fix test case handling of RUF014
zanieb d94dfd7
Fix preview mode messages
zanieb e7f095a
Deprecate `RuleSelector::Nursery`
zanieb b09fbb9
Deprecate `RuleGroup::Nursery`
zanieb b374802
Fix deprecated usage in test
zanieb 4ea6f17
Improve prefix comments
zanieb 2cab8b7
Fix unresolved merge conflict
zanieb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
Rename a bunch of "nursery" references to "preview"
- Loading branch information
commit 9b4aee105690765e6b5fbb467588fdbd168ad552
There are no files selected for viewing
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we use
PreviewMode
here?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The rule is in preview and is enabled or disabled by preview mode :D I want to think a bit more about "preview" vs "preview mode" in general so thanks for pointing this out.