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

Reduced the size of some lines in backend_test.py for PEP8 E501 #11184

Closed
wants to merge 2 commits into from
Closed

Reduced the size of some lines in backend_test.py for PEP8 E501 #11184

wants to merge 2 commits into from

Conversation

gabrieldemarmiesse
Copy link
Contributor

Summary

I first take on reducing the length of the lines in the backend_tests.py. Not everything is done. But I didn't want to make a PR which would be too big to simplify the review process.

Related Issues

PR Overview

Quite some tests have been parametrized to make the lines shorter and improve readability.

  • This PR requires new unit tests [y/n] (make sure tests are included)
  • This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
  • This PR is backwards compatible [y/n]
  • This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

@gabrieldemarmiesse gabrieldemarmiesse changed the title Reduced the size of some lines in backend_test.py. Reduced the size of some lines in backend_test.py for PEP8 E501 Oct 4, 2018
@gabrieldemarmiesse
Copy link
Contributor Author

Closing in favor of #11411

@gabrieldemarmiesse gabrieldemarmiesse deleted the line_length_backend branch November 7, 2018 17:05
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

1 participant