Skip to content

Validate Component parameter visibility analyzer #12479

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.

Sign up for GitHub

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

Jump to bottom
Closed
NTaylorMullen opened this issue Jul 23, 2019 · 4 comments
Closed

Validate Component parameter visibility analyzer #12479

NTaylorMullen opened this issue Jul 23, 2019 · 4 comments
Assignees
@NTaylorMullen
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates Done This issue has been fixed Validation This issue is used to track validation efforts
Milestone
3.0.0-preview8

Comments

@NTaylorMullen
Copy link

Need to verify:

  • Non-public component parameters result in warning
  • Non-public component parameter setter results in warning
  • Component parameter usage outside of a component results in warning
  • Component analyzers are auto-included in Blazor clientside projects
  • Component analyzers are auto-include in Blazor serverside projects

#8825
@NTaylorMullen NTaylorMullen added the Validation This issue is used to track validation efforts label Jul 23, 2019
@NTaylorMullen NTaylorMullen added this to the 3.0.0-preview8 milestone Jul 23, 2019
@NTaylorMullen NTaylorMullen self-assigned this Jul 23, 2019
@NTaylorMullen NTaylorMullen mentioned this issue Jul 23, 2019
Closed
@mkArtakMSFT mkArtakMSFT added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Jul 23, 2019
@NTaylorMullen
Copy link
Author

It doesn't detect increment/decrement operations: #12543

@NTaylorMullen
Copy link
Author

Got a fix out for this: #12544. Would love to pull this into preview8

@NTaylorMullen
Copy link
Author

#12544 was merged for preview8

@NTaylorMullen NTaylorMullen added the Done This issue has been fixed label Jul 25, 2019
@NTaylorMullen
Copy link
Author

Verified

@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NTaylorMullen NTaylorMullen

Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates Done This issue has been fixed Validation This issue is used to track validation efforts
Projects
None yet
Milestone
3.0.0-preview8
Development

No branches or pull requests
2 participants
@NTaylorMullen @mkArtakMSFT