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

Fix "Prefer format() over string interpolation operator" issue #9

Merged
merged 3 commits into from
Jun 9, 2017

Conversation

nanorepublica
Copy link
Contributor

This pull request automatically fixes all occurrences of the following issue:

Issue type: Prefer format() over string interpolation operator
Issue details: https://www.quantifiedcode.com/app/project/gh:founders4schools:duedilv3?groups=code_patterns/%3A4ACGxFj1

To adjust the commit message or the actual code changes, just rebase or cherry-pick the commits.
For questions or feedback reach out to cody@quantifiedcode.com.

Legal note: We won't claim any copyrights on the code changes.

Cheers,
Cody - Your code quality bot

@@ -28,13 +28,13 @@
f_list = []
for row in reader:
var = row['Field Name']
print '%s = "%s" # %s' % (var.upper(), var, row['Optional'])
print '{0!s} = "{1!s}" # {2!s}'.format(var.upper(), var, row['Optional'])

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E501 line too long (81 > 79 characters)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.735% when pulling 876245f on autofix/wrapped2_to3_fix into 18e7f39 on master.

@coveralls
Copy link

coveralls commented Jun 9, 2017

Coverage Status

Coverage increased (+1.1%) to 84.88% when pulling 2a49881 on autofix/wrapped2_to3_fix into 18e7f39 on master.

@founders4schools founders4schools deleted a comment from stickler-ci Jun 9, 2017
@founders4schools founders4schools deleted a comment from stickler-ci Jun 9, 2017
@founders4schools founders4schools deleted a comment from stickler-ci Jun 9, 2017
@founders4schools founders4schools deleted a comment from stickler-ci Jun 9, 2017
@founders4schools founders4schools deleted a comment from stickler-ci Jun 9, 2017
@founders4schools founders4schools deleted a comment from stickler-ci Jun 9, 2017
@coveralls
Copy link

coveralls commented Jun 9, 2017

Coverage Status

Coverage remained the same at 84.88% when pulling 93cce71 on autofix/wrapped2_to3_fix into 1031646 on master.

@nanorepublica nanorepublica merged commit 4004991 into master Jun 9, 2017
@nanorepublica nanorepublica deleted the autofix/wrapped2_to3_fix branch June 9, 2017 08:30
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.

None yet

5 participants