Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Py26 format #4263

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

Merged
merged 4 commits into from
Mar 23, 2015
Merged

Py26 format #4263

merged 4 commits into from
Mar 23, 2015

Conversation

montefra
Copy link
Contributor

fix few string formatting to avoid bugs with python 2.6. I hope that I got everything since grep [1] does not show me any match

[1] grep -r --include=*.py --exclude=.eggs/* -B 10 "\.format(" | grep {}

@mdboom
Copy link
Member

mdboom commented Mar 23, 2015

👍

tacaswell added a commit that referenced this pull request Mar 23, 2015
MNT : change format strings for py26 compatibility
@tacaswell tacaswell merged commit 04975c7 into matplotlib:master Mar 23, 2015
@tacaswell
Copy link
Member

Thank you!

@montefra
Copy link
Contributor Author

You're welcome. Thanks to grep and vim or was easy.

@WeatherGod
Copy link
Member

btw, you can use "git grep" to recursively grep the directories below you.
It will only search the files that are in the repository and not the other
files.

On Mon, Mar 23, 2015 at 2:10 AM, Francesco Montesano <
[email protected]> wrote:

You're welcome. Thanks to grep and vim or was easy.


Reply to this email directly or view it on GitHub
#4263 (comment)
.

@montefra
Copy link
Contributor Author

@WeatherGod I didn't know it. Thank you for the info!

@matthew-brett
Copy link
Contributor

git grep - nice!

@WeatherGod
Copy link
Member

people always complain to me about how unfriendly git is and how much they
loved subversion. Then I show them "git grep" and never hear a peep from
them again... :-P

On Mon, Mar 23, 2015 at 12:54 PM, Matthew Brett [email protected]
wrote:

git grep - nice!


Reply to this email directly or view it on GitHub
#4263 (comment)
.

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.

5 participants