Skip to content
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

Add configuration to the ModifierMissing rule #33

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

mrmans0n
Copy link
Owner

Adds configurability to the ModifierMissing rule, so that now users can configure whether they want non-public composables to be also subject to this check. The current configuration, where only public composables run this rule, keeps being the default. Values public_and_internal and all are added.

Fixes #25

Adds configurability to the ModifierMissing rule, so that now users can configure whether they want non-public composables to be also subject to this check.
The current configuration, where only public composables run this rule, keeps being the default. Values `public_and_internal` and `all` are added.

Fixes #25
@mrmans0n mrmans0n merged commit cfeb361 into main Mar 15, 2023
@mrmans0n mrmans0n deleted the nacho/configurable-modifier-missing branch March 15, 2023 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reasoning behind Modifiers being only mandatory in public functions
1 participant