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

Fix the syntax error for the constant array access #4684 #4735

Merged

Conversation

junichi11
Copy link
Member

#4684

  • Fix the grammar file
  • Add unit tests

Screenshot

Before:

nb-gh-4684-syntax-before

After:

nb-gh-4684-syntax

@junichi11 junichi11 added the PHP [ci] enable extra PHP tests (php/php.editor) label Oct 5, 2022
@junichi11 junichi11 added this to the NB16 milestone Oct 5, 2022
@ebarboni ebarboni linked an issue Oct 5, 2022 that may be closed by this pull request
- Fix the grammar file
- Add unit tests
@junichi11 junichi11 force-pushed the gh-issue-4684-const-array-syntax-error branch from 4181555 to ae7de61 Compare October 5, 2022 15:47
@matthiasblaesing
Copy link
Contributor

It seems github actions did not pick up the PHP label. I'll try the "super secret" trick that was added (locking + unlocking this PR). Please don't be irritated.

@apache apache locked and limited conversation to collaborators Oct 5, 2022
@apache apache unlocked this conversation Oct 5, 2022
@junichi11 junichi11 requested a review from tmysik October 6, 2022 01:58
@junichi11
Copy link
Member Author

@tmysik Could you please have a look at this? If there is no problem, let's merge it :) Thanks!

@tmysik tmysik merged commit 36441f7 into apache:master Oct 7, 2022
@junichi11 junichi11 deleted the gh-issue-4684-const-array-syntax-error branch October 7, 2022 23:29
@junichi11
Copy link
Member Author

@tmysik Thank you for your review!

@tmysik
Copy link
Member

tmysik commented Oct 8, 2022

@junichi11 Nothing to thank for, really. Thanks for your work!

@junichi11 junichi11 added the hacktoberfest-accepted Hacktoberfest label Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Hacktoberfest PHP [ci] enable extra PHP tests (php/php.editor)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PHP] Parser syntax error - accessing a const array value
3 participants