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

Fix caching issue caused by incorrect isProvisional check #16989

Merged
merged 1 commit into from
Feb 24, 2023

Conversation

smarter
Copy link
Member

@smarter smarter commented Feb 21, 2023

A static TypeRef can still be provisional if it's currently being completed (see the logic in Namer#TypeDefCompleter#typeSig).

Fixes #16950.

@smarter smarter added the backport:nominated If we agree to backport this PR, replace this tag with "backport:accepted", otherwise delete it. label Feb 21, 2023
@smarter smarter requested a review from odersky February 21, 2023 20:54
A static TypeRef can still be provisional if it's currently being completed (see
the logic in `Namer#TypeDefCompleter#typeSig`).

Fixes scala#16950.
@odersky odersky merged commit 1e981ab into scala:main Feb 24, 2023
@odersky odersky deleted the fix-i16590 branch February 24, 2023 18:11
@Kordyjan Kordyjan added backport:accepted This PR needs to be backported, once it's been backported replace this tag by "backport:done" and removed backport:nominated If we agree to backport this PR, replace this tag with "backport:accepted", otherwise delete it. labels Apr 13, 2023
@Kordyjan Kordyjan added backport:done This PR was successfully backported. and removed backport:accepted This PR needs to be backported, once it's been backported replace this tag by "backport:done" labels Apr 17, 2023
@Kordyjan Kordyjan modified the milestones: 3.3.1, 3.3.0 Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:done This PR was successfully backported.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typer: Instability & Inconsistent Conformance Check Report
3 participants