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.

Sign up for GitHub

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

Jump to bottom

Template parameter with constraint on itself should error #2788

Closed
timotheeguerin opened this issue Jan 11, 2024 · 1 comment · Fixed by #2932
Closed

Template parameter with constraint on itself should error #2788

timotheeguerin opened this issue Jan 11, 2024 · 1 comment · Fixed by #2932
Labels
bug Something isn't working triaged:core
Milestone
[2024] March

Comments

@timotheeguerin
Copy link
Member

timotheeguerin commented Jan 11, 2024 •
edited
Loading 

model Test<A extends A> {
  
}

Playground

Typescript does this
image

Same with circular reference

model Test<A extends B, B extends A> {
  
}
@markcowl markcowl added the bug Something isn't working label Jan 11, 2024
@markcowl markcowl added this to the [2024] February milestone Jan 11, 2024
@markcowl
Copy link
Contributor

pri: 2

@timotheeguerin timotheeguerin mentioned this issue Feb 16, 2024
Merged
timotheeguerin added a commit that referenced this issue Feb 20, 2024
@timotheeguerin
Report error when having a circular template constraint (#2932)
09aae4c 
fix [#2788](#2788)
markcowl pushed a commit to markcowl/cadl that referenced this issue Mar 5, 2024
@timotheeguerin @markcowl
Report error when having a circular template constraint (microsoft#2932)
221bc8b 
fix [microsoft#2788](microsoft#2788)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working triaged:core
Projects
None yet
Milestone
[2024] March
Development

Successfully merging a pull request may close this issue.

Report error when having a circular template constraint timotheeguerin/typespec
2 participants
@timotheeguerin @markcowl