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 message arguments to property-disallowed-list #6462

Closed
chloerice opened this issue Nov 10, 2022 · 0 comments · Fixed by #6463
Closed

Add message arguments to property-disallowed-list #6462

chloerice opened this issue Nov 10, 2022 · 0 comments · Fixed by #6463
Labels
status: ready to implement is ready to be worked on by someone type: enhancement a new feature that isn't related to rules

Comments

@chloerice
Copy link
Contributor

What is the problem you're trying to solve?

In the same vein as #6453, #6454, and #6455, I'd love to be able to access the name of the property that's rejected by property-disallowed-list so that I can map it to a custom message and relevant documentation link.

What solution would you like to see?

report({
- message: messages.rejected(prop),
+ message: messages.rejected,
+ messageArgs: [prop],
  word: prop,
  node: decl,
  result,
  ruleName,
});
@ybiquitous ybiquitous added status: ready to implement is ready to be worked on by someone type: enhancement a new feature that isn't related to rules labels Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready to implement is ready to be worked on by someone type: enhancement a new feature that isn't related to rules
Development

Successfully merging a pull request may close this issue.

2 participants