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 "Explicitly number replacement fields in a format string" issue #10

Merged
merged 1 commit into from
Jun 9, 2017

Conversation

nanorepublica
Copy link
Contributor

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

Issue type: Explicitly number replacement fields in a format string
Issue details: https://www.quantifiedcode.com/app/project/gh:founders4schools:duedilv3?groups=code_patterns/%3A3aSZNLMu

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

@@ -86,7 +86,7 @@ def set_api(self, api_key=None, sandbox=False):
try:
self.base_url = API_URLS.get(self.api_type, 'lite')
except AttributeError:
raise ValueError('Duedil API type must be "{}"'.format('", "'.join(API_URLS.keys())))
raise ValueError('Duedil API type must be "{0}"'.format('", "'.join(API_URLS.keys())))

Choose a reason for hiding this comment

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

E501 line too long (98 > 79 characters)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.735% when pulling 5ea3def on autofix/wrapped2_to3_fix-0 into 18e7f39 on master.

@coveralls
Copy link

coveralls commented Jun 9, 2017

Coverage Status

Coverage remained the same at 84.88% when pulling c3128ba on autofix/wrapped2_to3_fix-0 into 4004991 on master.

@nanorepublica nanorepublica merged commit 8fa04d7 into master Jun 9, 2017
@nanorepublica nanorepublica deleted the autofix/wrapped2_to3_fix-0 branch June 9, 2017 08:40
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