Skip to content

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