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

Update AnalyzerBannedSymbols.txt #6882

Merged
merged 1 commit into from
Aug 21, 2023
Merged

Update AnalyzerBannedSymbols.txt #6882

merged 1 commit into from
Aug 21, 2023

Conversation

jaredpar
Copy link
Member

Prevent future uses of System.Random in analyzers / generators

@jaredpar jaredpar requested a review from a team as a code owner August 21, 2023 20:22
@codecov
Copy link

codecov bot commented Aug 21, 2023

Codecov Report

Merging #6882 (85a3139) into main (76d99c5) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6882      +/-   ##
==========================================
- Coverage   96.39%   96.39%   -0.01%     
==========================================
  Files        1403     1403              
  Lines      330977   330977              
  Branches    10890    10890              
==========================================
- Hits       319054   319051       -3     
- Misses       9190     9191       +1     
- Partials     2733     2735       +2     

@jaredpar jaredpar merged commit 60872a8 into main Aug 21, 2023
13 of 14 checks passed
@jaredpar jaredpar deleted the dev/jaredpar/ban-random branch August 21, 2023 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants