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

Can add more rules for the same call to have different messages for various params #232

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

spaze
Copy link
Owner

@spaze spaze commented Dec 13, 2023

The rules have to be sorted from the most specific to the most general to have different messages for different params, see FunctionCallsParamsMessagesTest.

This makes sense mostly|only when you want to show a different message when various params have been disallowed. For which the new config options allowExceptParamsAnyValue and disallowParamsAnyValue (which is an alias for the former) have been added.

Close #155

…arious params.

The rules have to be sorted from the most specific to the most general to have different messages for different params, see FunctionCallsParamsMessagesTest.

Close #155
@spaze spaze self-assigned this Dec 13, 2023
@spaze spaze merged commit ee8ec5f into main Dec 15, 2023
74 checks passed
@spaze spaze deleted the spaze/different-errors-for-disallowed-params branch December 15, 2023 03:06
@spaze
Copy link
Owner Author

spaze commented Dec 22, 2023

Released in 3.0.0.

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.

When a call is disallowed due to a parameter rule, display a different message
1 participant