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

Rename all remaining properties to camelCase #1043

Merged
merged 8 commits into from
Aug 14, 2022
Merged

Conversation

mvorisek
Copy link
Member

@mvorisek mvorisek commented Jul 30, 2022

finish #1040

rename all remaining properties with underscore in name: (public|protected|private) .*\$\w+_

regex to fix code to update: (id|title|our|their|master|foreign)_field

@mvorisek mvorisek force-pushed the strong_type_title_field branch 2 times, most recently from 5dc3e34 to 4bb1e2e Compare August 5, 2022 09:32
@mvorisek mvorisek force-pushed the strong_type_title_field branch 2 times, most recently from 4bd7f41 to e1c659b Compare August 14, 2022 15:08
@mvorisek mvorisek changed the title Strong type Model::$title_field Rename all remaining properties to camelCase Aug 14, 2022
@mvorisek mvorisek marked this pull request as ready for review August 14, 2022 17:44
@mvorisek mvorisek added the RTM label Aug 14, 2022
@mvorisek mvorisek merged commit 5b3a93e into develop Aug 14, 2022
@mvorisek mvorisek deleted the strong_type_title_field branch August 14, 2022 18:08
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.

1 participant