Skip to content
This repository was archived by the owner on Nov 1, 2023. It is now read-only.

Apply allowlist to source conversion#3208

Merged
AdamL-Microsoft merged 6 commits into
microsoft:mainfrom
ranweiler:source-convert-allowlist
Jun 21, 2023
Merged

Apply allowlist to source conversion#3208
AdamL-Microsoft merged 6 commits into
microsoft:mainfrom
ranweiler:source-convert-allowlist

Conversation

@ranweiler

Copy link
Copy Markdown
Contributor

Closes #3207.

@codecov-commenter

codecov-commenter commented Jun 21, 2023

Copy link
Copy Markdown

Codecov Report

Merging #3208 (cc91898) into main (a720dd1) will decrease coverage by 3.91%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3208      +/-   ##
==========================================
- Coverage   29.53%   25.63%   -3.91%     
==========================================
  Files         332      152     -180     
  Lines       39060    16289   -22771     
==========================================
- Hits        11535     4175    -7360     
+ Misses      27525    12114   -15411     

see 180 files with indirect coverage changes

@ranweiler

Copy link
Copy Markdown
Contributor Author

Manually inspected source coverage reports in integration tests.

@AdamL-Microsoft AdamL-Microsoft merged commit fda68ac into microsoft:main Jun 21, 2023
@ranweiler ranweiler deleted the source-convert-allowlist branch June 21, 2023 19:41
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jun 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Excluded source files included in source coverage if inlined

4 participants