Skip to content

Conversation

LiamPattinson
Copy link
Collaborator

Adds a basic implementation for #106, so you can now specify the full names of rules:

fortitude check --select=implicit-typing

Also works in settings files.

Missing some features:

  • Can't specify categories by full name, so --select=typing doesn't work.
  • Human-readable names are mapped 1-to-1 with rule codes. In the future we'll likely need multiple human-readable names and multiple codes mapping to each rule.

Ultimately I think we'll need to add more macro machinery to do it properly, but right now I'm still having issues figuring out how everything fits together.

Acronyms in upper-camel-case should count as one word, so
IOError has been renamed IoError, and ASTRule has been
renamed AstRule.

This will help convert cases back and forth so users can
select rules by name.
Copy link
Member

@ZedThree ZedThree left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing! 🎉

@ZedThree ZedThree merged commit ead4ca7 into main Nov 14, 2024
22 checks passed
@ZedThree ZedThree deleted the select_rules_by_name branch November 14, 2024 14:14
@ZedThree ZedThree added the cli Related to the command line interface label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Related to the command line interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants