Skip to content

rename methods to snake_case - #265

Merged
leandrotoledo merged 12 commits into
masterfrom
snakes
May 14, 2016
Merged

rename methods to snake_case#265
leandrotoledo merged 12 commits into
masterfrom
snakes

Conversation

@rahiel

@rahiel rahiel commented Apr 28, 2016

Copy link
Copy Markdown
Contributor

This should solve #259 while keeping backward compatibility for users who don't like to change their existing code. This also fixes #255

@tsnoam

tsnoam commented Apr 28, 2016

Copy link
Copy Markdown
Member

@rahiel imo, this should issue deprecation warning at first and at a later release remove the backward compatibility

@rahiel rahiel changed the title add snake_case aliases for CamelCase methods rename methods to snake_case Apr 28, 2016
@rahiel

rahiel commented Apr 28, 2016

Copy link
Copy Markdown
Contributor Author

@rahiel imo, this should issue deprecation warning at first and at a later release remove the backward compatibility

Done! I might have missed some internal invocations the old methods.

Did I miss any other methods?

@rahiel rahiel added this to the 5.0 milestone Apr 28, 2016
@jh0ker

jh0ker commented Apr 28, 2016

Copy link
Copy Markdown
Member

Updating examples should be done on release of this, yes?

@rahiel

rahiel commented Apr 28, 2016

Copy link
Copy Markdown
Contributor Author

Updating examples should be done on release of this, yes?

Yes, I think we should quickly make a new release with the new interface so new users don't have to adapt later.

@tsnoam

tsnoam commented Apr 28, 2016

Copy link
Copy Markdown
Member

@rahiel I think it must come out together with the fixes in my PR from yesterday. I dearly hope to have the time to fix the merge tonight.

@rahiel rahiel added the 📋 pending-review work status: pending-review label Apr 30, 2016
@leandrotoledo

Copy link
Copy Markdown
Member

Reloading PyPy tests.

@leandrotoledo

Copy link
Copy Markdown
Member

Should we get this merged?

@jh0ker

jh0ker commented May 14, 2016

Copy link
Copy Markdown
Member

@leandrotoledo LGTM

@leandrotoledo
leandrotoledo merged commit 56b1d4f into master May 14, 2016
@leandrotoledo
leandrotoledo deleted the snakes branch May 14, 2016 13:56
@leandrotoledo leandrotoledo modified the milestones: 4.1, 5.0 May 15, 2016
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 25, 2020
@Bibo-Joshi Bibo-Joshi added 🔌 enhancement pr description: enhancement and removed enhancement labels Nov 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🔌 enhancement pr description: enhancement 📋 pending-review work status: pending-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PEP8 issue: Methods should be snake case

5 participants