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

More f-strings #8178

Merged
merged 2 commits into from Aug 17, 2019

Conversation

@asherf
Copy link
Contributor

commented Aug 16, 2019

Problem

This code was using str.format()

Solution

Switch it to use f-strings

@asherf asherf changed the title ### Problem More f-strings Aug 16, 2019

@Eric-Arellano
Copy link
Contributor

left a comment

Yay!

src/python/pants/util/xml_parser.py Outdated Show resolved Hide resolved
src/python/pants/util/xml_parser.py Outdated Show resolved Hide resolved
src/python/pants/util/xml_parser.py Outdated Show resolved Hide resolved

@Eric-Arellano Eric-Arellano merged commit b4e316f into pantsbuild:master Aug 17, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
patliu85 added a commit to twitter/pants that referenced this pull request Aug 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.