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

Tech debt: Complete "Collect diags instead of returning directly" #37569

Open
ewbankkit opened this issue May 16, 2024 · 1 comment
Open

Tech debt: Complete "Collect diags instead of returning directly" #37569

ewbankkit opened this issue May 16, 2024 · 1 comment
Labels
technical-debt Addresses areas of the codebase that need refactoring or redesign.

Comments

@ewbankkit
Copy link
Contributor

ewbankkit commented May 16, 2024

Complete the work to collect diagnostics in diags instead of returning error diagnostic directly.
This ensures that warning diagnostics are not accidentally dropped.
We need to fix services n through z.

See .ci/semgrep/pluginsdk/diags.yml.

Relates #34837.
Relates #34727.
Relates #34695.
Relates #34620.
Relates #34526.

Copy link

Community Note

Voting for Prioritization

  • Please vote on this issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize this request.
  • Please see our prioritization guide for information on how we prioritize.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

Volunteering to Work on This Issue

  • If you are interested in working on this issue, please leave a comment.
  • If this would be your first contribution, please review the contribution guide.

@ewbankkit ewbankkit added the technical-debt Addresses areas of the codebase that need refactoring or redesign. label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical-debt Addresses areas of the codebase that need refactoring or redesign.
Projects
None yet
Development

No branches or pull requests

1 participant