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

Clean up suppression of CS0109 #249

Closed
wants to merge 3 commits into from

Conversation

Balkoth
Copy link

@Balkoth Balkoth commented Oct 5, 2023

Fixes

#248

PR Type

  • Bugfix

What is the current behavior?

Suppresses CS0109 on multiple occasions.

What is the new behavior?

Removed the suppresions and adjusted the code accordingly.

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • Tested code with current supported SDKs
  • Contains NO breaking changes

@michael-hawker
Copy link
Member

Thanks @Balkoth! I replied in the issue with more details.

@Arlodotexe we need to figure out why the verbosity is bring ignored or what's ballooning the logs to where we can't see the errors in the CI easily anymore. Would be nice to be able to get at those without having to download the logs and search.

@Arlodotexe Arlodotexe linked an issue Jan 23, 2024 that may be closed by this pull request
24 tasks
@Arlodotexe Arlodotexe self-requested a review January 23, 2024 21:55
@Arlodotexe
Copy link
Member

Looks like we're not able to clean up this suppression yet. I've filed a ticket on Uno's side to see what we can do (see here)

@Arlodotexe Arlodotexe closed this Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Clean up suppression of CS0109 and CS0114
4 participants