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

Array model allows properties to be defined #2758

Closed
timotheeguerin opened this issue Dec 15, 2023 · 0 comments · Fixed by #2873
Closed

Array model allows properties to be defined #2758

timotheeguerin opened this issue Dec 15, 2023 · 0 comments · Fixed by #2873
Assignees
@timotheeguerin
Labels
bug Something isn't working triaged:core
Milestone
[2024] February

Comments

@timotheeguerin
Copy link
Member 

model WorksWithIs is Array<string> {
  prop: string;
}

model WorksWithExtends extends Array<string> {
  prop: string;
}

model FailBecauseType is Array<string> {
  prop: int32;
}

I feel like this used to fail before

Playground
@markcowl markcowl added bug Something isn't working and removed needs-area labels Dec 18, 2023
@markcowl markcowl modified the milestones: Backlog, [2024] February Dec 18, 2023
@timotheeguerin timotheeguerin self-assigned this Feb 1, 2024
@timotheeguerin timotheeguerin mentioned this issue Feb 1, 2024
Merged
timotheeguerin added a commit that referenced this issue Feb 5, 2024
@timotheeguerin @markcowl
Fix: Error out when using properties in array model (#2873)
fd4fdfb 
fix #2758

---------

Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timotheeguerin timotheeguerin

Labels
bug Something isn't working triaged:core
Projects
None yet
Milestone
[2024] February
Development

Successfully merging a pull request may close this issue.

Fix: Error out when using properties in array model timotheeguerin/typespec
2 participants
@timotheeguerin @markcowl