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

Make one comment in the PR and keep updating it #17

Merged
merged 2 commits into from
Feb 20, 2017
Merged

Conversation

OrkoHunter
Copy link
Collaborator

Closes #16

Thanks @glemaitre for the suggestion.

@pep8speaks
Copy link
Contributor

Hello @OrkoHunter! Thanks for submitting the PR.

Line 51:9: E123 closing bracket does not match indentation of opening bracket's line
Line 53:8: E221 multiple spaces before operator
Line 463:17: E123 closing bracket does not match indentation of opening bracket's line

Copy link
Collaborator Author

@OrkoHunter OrkoHunter left a comment

Choose a reason for hiding this comment

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

@pep8speaks suggest diff

@pep8speaks
Copy link
Contributor

Here you go with the gist !

You can ask me to create a PR against this branch with those fixes. Submit a review comment as @pep8speaks pep8ify.

@OrkoHunter

@pep8speaks
Copy link
Contributor

Hello @OrkoHunter! Thanks for updating the PR.

@OrkoHunter OrkoHunter merged commit 5141cbe into master Feb 20, 2017
@OrkoHunter OrkoHunter deleted the single_comment branch February 20, 2017 15:52
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.

2 participants