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

Bounded type parameters with export #18569

Closed
SimonGuilloud opened this issue Sep 18, 2023 · 1 comment · Fixed by #18647
Closed

Bounded type parameters with export #18569

SimonGuilloud opened this issue Sep 18, 2023 · 1 comment · Fixed by #18647

Comments

@SimonGuilloud
Copy link

Compiler version

3.3.1, 3.2.1

Minimized code

trait M1 {
  trait A
  trait F[T<:A]
}

object M2 extends M1

trait Test{
  export M2.*
  def y: F[A]
}

Output

Type argument Test.this.A does not conform to upper bound M1.this.A

Expectation

No error.

If the bound T<:A is in a function rather than a trait, compiles. If export is replaced by import, compiles. If F is defined in M2, compiles. If F is a class rather than a trait, does not compile

@SimonGuilloud SimonGuilloud added itype:bug stat:needs triage Every issue needs to have an "area" and "itype" label labels Sep 18, 2023
@Kordyjan Kordyjan added area:export and removed stat:needs triage Every issue needs to have an "area" and "itype" label labels Sep 19, 2023
@mbovel mbovel added the Spree Suitable for a future Spree label Sep 26, 2023
@scala-center-bot
Copy link

This issue was picked for the Issue Spree No. 37 of 03 October 2023 which takes place in 7 days. @mbovel, @iusildra, @hamzaremmal, @jan-pieter will be working on it. If you have any insight into the issue or guidance on how to fix it, please leave it here.

@dwijnand dwijnand linked a pull request Oct 4, 2023 that will close this issue
@mbovel mbovel removed the Spree Suitable for a future Spree label Jan 14, 2024
@Kordyjan Kordyjan added this to the 3.4.2 milestone Mar 28, 2024
@Kordyjan Kordyjan modified the milestones: 3.4.2, 3.5.0 May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants