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

Allow trailing commas in array #2284

Closed
timotheeguerin opened this issue Aug 10, 2023 · 0 comments · Fixed by #4837 · May be fixed by #4373
Closed

Allow trailing commas in array #2284

timotheeguerin opened this issue Aug 10, 2023 · 0 comments · Fixed by #4837 · May be fixed by #4373
Assignees
@timotheeguerin
Labels
triaged:core
Milestone
Backlog

Comments

@timotheeguerin
Copy link
Member

timotheeguerin commented Aug 10, 2023 •
edited
Loading 

alias Tasks = [
  "foo",
       ^ currently error out
];

Like for many other things were we support trailing ocmmas for multi line this would be nice

Playground Link
@markcowl markcowl added this to the Backlog milestone Aug 14, 2023
@timotheeguerin timotheeguerin self-assigned this Sep 9, 2024
@timotheeguerin timotheeguerin mentioned this issue Sep 9, 2024
Open
@timotheeguerin timotheeguerin mentioned this issue Oct 23, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Nov 21, 2024
@timotheeguerin
Allow trailing delimited in most locations (#4837)
8afd0e8 
fix #2284 
fix #809

Extracted the parser change from pr #4372 and made it more generic to
cover also issue #809
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timotheeguerin timotheeguerin

Labels
triaged:core
Projects
None yet
Milestone
Backlog
2 participants
@timotheeguerin @markcowl