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

Allow short array syntax in our projects #36

Merged
merged 1 commit into from
Nov 25, 2019
Merged

Conversation

grappler
Copy link
Contributor

This was included in WordPress-Core ruleset WordPress/WordPress-Coding-Standards#1770

@ocean90
Copy link
Member

ocean90 commented Nov 25, 2019

Should we use a fixed version in https://github.com/wearerequired/coding-standards/blob/master/composer.json#L16 so rules are only used when we have reviewed anything new?

@grappler grappler merged commit 192ceb3 into master Nov 25, 2019
@delete-merged-branch delete-merged-branch bot deleted the AllowShortArraySyntax branch November 25, 2019 10:20
@grappler
Copy link
Contributor Author

Should we use a fixed version so rules are only used when we have reviewed anything new?

I think that should be a good idea. We just need to remember to regularly update and test the version.

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.

2 participants