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

Illustrate unexpected oblivious on implicit containing type in public API #4681

Merged
merged 1 commit into from
Jan 11, 2021

Conversation

jcouv
Copy link
Member

@jcouv jcouv commented Jan 9, 2021

Adding a couple of tests to illustrate a compiler issue that's affecting the publicAPI analyzer (complaining about oblivious in code that is seemingly fully nullable-enabled).
Taggging @mavasani for review.

@jcouv jcouv self-assigned this Jan 9, 2021
@jcouv jcouv requested a review from a team as a code owner January 9, 2021 00:15
@codecov
Copy link

codecov bot commented Jan 9, 2021

Codecov Report

Merging #4681 (a6f8400) into master (d8bf975) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #4681      +/-   ##
==========================================
- Coverage   95.79%   95.79%   -0.01%     
==========================================
  Files        1193     1193              
  Lines      270350   270395      +45     
  Branches    16300    16302       +2     
==========================================
+ Hits       258982   259014      +32     
+ Misses       9289     9288       -1     
- Partials     2079     2093      +14     

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.

None yet

2 participants