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

Make all options snake_case #4634

Merged
merged 1 commit into from Nov 23, 2019
Merged

Make all options snake_case #4634

merged 1 commit into from Nov 23, 2019

Conversation

kubawerlos
Copy link
Contributor

Resolves #4389

@julienfalque julienfalque added this to the 2.17.0 milestone Nov 9, 2019
@keradus
Copy link
Member

keradus commented Nov 23, 2019

Thank you @kubawerlos.

keradus added a commit that referenced this pull request Nov 23, 2019
@keradus
minor #4634 Make all options snake_case (kubawerlos)
a4513fa 
This PR was merged into the 2.17-dev branch.

Discussion
----------

Make all options snake_case

Resolves #4389

Commits
-------

369a7b1 Make all options snake_case
@keradus keradus merged commit 369a7b1 into PHP-CS-Fixer:master Nov 23, 2019
@keradus
Copy link
Member

keradus commented Nov 23, 2019

@kubawerlos , can you please raise a PR for 3.0 line to remove those deprecated options ?

@kubawerlos kubawerlos deleted the ensure-all-options-are-snake-case branch November 24, 2019 06:47
@kubawerlos kubawerlos mentioned this pull request Apr 16, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@julienfalque julienfalque julienfalque approved these changes

@GrahamCampbell GrahamCampbell GrahamCampbell approved these changes

Assignees
No one assigned
Labels
kind/enhancement
Projects
None yet
Milestone
2.17.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@kubawerlos @keradus @julienfalque @GrahamCampbell