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.

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

Update S5773: add a message on secondary locations #3694

Closed
nicolas-harraudeau-sonarsource opened this issue Oct 29, 2020 · 0 comments · Fixed by #8224
Closed

Update S5773: add a message on secondary locations #3694

nicolas-harraudeau-sonarsource opened this issue Oct 29, 2020 · 0 comments · Fixed by #8224
Assignees
@costin-zaharia-sonarsource
Labels
Area: C# C# rules related issues. Sprint: Hardening Fix FPs/FNs/improvements
Milestone
9.13

Comments

@nicolas-harraudeau-sonarsource
Copy link

nicolas-harraudeau-sonarsource commented Oct 29, 2020 •
edited
Loading

RSPEC-5773 creates a secondary location on the function allowing all types to be deserialized. This secondary locations is very useful to understand the issue but without a message it is hard to grasp at first. We can improve the user experience by adding a message.

The secondary location message is already provided in the RSPEC.
@nicolas-harraudeau-sonarsource nicolas-harraudeau-sonarsource changed the title Update S4143: add a message on secondary locations Update S5773: add a message on secondary locations Oct 29, 2020
@costin-zaharia-sonarsource costin-zaharia-sonarsource added the Sprint: Hardening Fix FPs/FNs/improvements label Oct 13, 2023
Merged
@sebastien-marichal sebastien-marichal added this to the 9.13 milestone Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@costin-zaharia-sonarsource costin-zaharia-sonarsource

Labels
Area: C# C# rules related issues. Sprint: Hardening Fix FPs/FNs/improvements
Projects
None yet
Milestone
9.13
Development

Successfully merging a pull request may close this issue.

S5773: Add message for secondary locations SonarSource/sonar-dotnet
3 participants
@nicolas-harraudeau-sonarsource @costin-zaharia-sonarsource @sebastien-marichal