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

[PHP] Add support for enum in class-name_feature context #2752

Merged
merged 1 commit into from Feb 18, 2021
Merged

[PHP] Add support for enum in class-name_feature context #2752

merged 1 commit into from Feb 18, 2021

Conversation

Ayesh
Copy link
Contributor

@Ayesh Ayesh commented Feb 6, 2021

PHP 8.1 adds Enum support. This PR updates the class-like structure matching regex and its tests to accommodate the new enum structures.

Related: #2731

[PHP 8.1 adds Enum support](https://php.watch/versions/8.1/enums#enum-syntax). This PR updates the class-like structure matching regex and its tests to accommodate the new `enum` structures.

Related: #2731
@github-actions
Copy link

github-actions bot commented Feb 6, 2021

JS File Size Changes (gzipped)

A total of 1 files have changed, with a combined diff of +7 B (+0.4%).

file master pull size diff % diff
components/prism-php.min.js 1.86 KB 1.87 KB +7 B +0.4%

Generated by 🚫 dangerJS against 759670c

@Ayesh
Copy link
Contributor Author

Ayesh commented Feb 6, 2021

I will keep this PR as a draft until the voting period for Enum proposal ends. It is very likely to pass, but I'd like to wait for the final vote off. Thank you.

Copy link
Member

@RunDevelopment RunDevelopment left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM;
Looking forward to having this in Prism once the draft is through.

@Ayesh
Copy link
Contributor Author

Ayesh commented Feb 18, 2021

RFC vote ended with 44:7 in favor. PHP 8.1 will have Enums 🎉🎊.

@Ayesh Ayesh marked this pull request as ready for review February 18, 2021 08:46
@RunDevelopment RunDevelopment merged commit f79b0ee into PrismJS:master Feb 18, 2021
@RunDevelopment
Copy link
Member

Thank you for contributing @Ayesh!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants