-
Notifications
You must be signed in to change notification settings - Fork 783
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
Implicit constructors should only target AttributeTargets.Constructor #16845
Conversation
❗ Release notes required
|
This should be merged after #16790 |
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.
Thanks, good stuff again!
Will fix the conflicts tomorrow :) |
Sure, thanks :) |
@psfinaki Based on your previous suggestion. I merged all the EnforceAttributeTargets LanguageFeatures into one. |
This is ready |
Awesome, thanks :) |
I have 2-3 more PR planned in the upcoming weeks |
Description
Fixes #16794
Before
After
Error 842, "This attribute is not valid for use on this language element")
Checklist
Test cases added
Release notes entry updated: