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 Travis CI build error caused by previous commit #289

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

ipfilip
Copy link
Contributor

@ipfilip ipfilip commented Sep 7, 2022

  • This contribution adheres to CONTRIBUTING.md.
  • New tests have been created for any new features or regression tests for bugfixes.
  • tox successfully runs, including unit tests and style checks (see CONTRIBUTING.md).

What does this Pull Request accomplish?

Fixing Travis CI errors in my previously merged pull request.

Cosmetic changes to deal with two flake8 objections and a little update to .travis.yml config to avoid an error caused by setuptools (see pypa/setuptools#3293).

Why should this Pull Request be merged?

To fix build errors.

What testing has been done?

Tests run with no problem.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.005%) to 67.546% when pulling aadf295 on ipfilip:main into 8fe225b on ni:main.

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