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

List and list spread as jsx children cause syntax error #1467

Open
chenglou opened this issue Oct 6, 2017 · 0 comments · May be fixed by #1972
Open

List and list spread as jsx children cause syntax error #1467

chenglou opened this issue Oct 6, 2017 · 0 comments · May be fixed by #1972
Labels
Error messages confusing, missing, or just weak error messages

Comments

@chenglou
Copy link
Member

chenglou commented Oct 6, 2017

We can ship this after the new release, since nobody uses list as children for now.

Should probably have some error message in the meantime though
@chenglou chenglou mentioned this issue Oct 14, 2017
Closed
7 tasks
@chenglou chenglou mentioned this issue Nov 22, 2017
Closed
16 tasks
anmonteiro added a commit to anmonteiro/reason that referenced this issue May 29, 2018
@anmonteiro
Parse list and list spread in JSX
aeebd5a 
fixes reasonml#1467
@anmonteiro anmonteiro linked a pull request May 29, 2018 that will close this issue
Open
anmonteiro added a commit to anmonteiro/reason that referenced this issue Jun 3, 2018
@anmonteiro
Parse list and list spread in JSX
9d77da3 
fixes reasonml#1467
@jaredly jaredly added the Error messages confusing, missing, or just weak error messages label Jun 14, 2018
anmonteiro added a commit to anmonteiro/reason that referenced this issue Aug 9, 2018
@anmonteiro
Parse list and list spread in JSX
0217c11 
fixes reasonml#1467
anmonteiro added a commit to anmonteiro/reason that referenced this issue Aug 9, 2018
@anmonteiro
Parse list and list spread in JSX
3bc3f91 
fixes reasonml#1467
Drup pushed a commit to Drup/reason that referenced this issue Feb 4, 2020
@anmonteiro @Drup
Parse list and list spread in JSX
bd34f6c 
fixes reasonml#1467
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Error messages confusing, missing, or just weak error messages
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Parse list and list spread in JSX anmonteiro/reason
2 participants
@jaredly @chenglou