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

Support hashbang #1417

Merged
merged 2 commits into from
Nov 9, 2023
Merged

Support hashbang #1417

merged 2 commits into from
Nov 9, 2023

Conversation

p-bakker
Copy link
Collaborator

@p-bakker p-bakker commented Nov 4, 2023

Closes #1274

Replaces #1341

@p-bakker p-bakker added this to the ES2023 milestone Nov 4, 2023
@p-bakker p-bakker force-pushed the hashbang-support branch 2 times, most recently from 04fe3be to 88e2ebf Compare November 4, 2023 18:24
@rbri
Copy link
Collaborator

rbri commented Nov 5, 2023

@p-bakker another minor step forward - great

overall it looks good, only a small comment, see above

@gbrail
Copy link
Collaborator

gbrail commented Nov 7, 2023

Makes sense to me too -- I'll try it out. Thanks!

@gbrail gbrail merged commit 85effa5 into mozilla:master Nov 9, 2023
3 checks passed
rbri pushed a commit to rbri/rhino that referenced this pull request Nov 10, 2023
Support hashbang syntax at the start of a script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support ES2023 hashbang grammer
3 participants