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(CI): Hold back flake8 version #155

Merged
merged 3 commits into from Jul 28, 2017
Merged

fix(CI): Hold back flake8 version #155

merged 3 commits into from Jul 28, 2017

Conversation

epage
Copy link
Contributor

@epage epage commented Jul 28, 2017

flake8 3.4 was recently released and broke support for ignoring multiple
items on a line
.

  • 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).

flake8 3.4 was recently released and [broke support for ignoring multiple
items on a line](https://gitlab.com/pycqa/flake8/issues/356).
Flake8 breaking nixnet is finally motivating me to switch our builds to
lock dependencies.  This is just the first step, moving the dependencies
into a dedicated file.
This has the benefit of pinning or locking every dependency so we have
stable builds and are easily upgraded (through piptools's pip-compile or
pyup) in a controlled way.
@coveralls
Copy link

coveralls commented Jul 28, 2017

Coverage Status

Coverage remained the same at 64.24% when pulling 7f044f7 on epage:lock into 13a2757 on ni:master.

@epage epage merged commit bc8096e into ni:master Jul 28, 2017
@epage epage deleted the lock branch July 28, 2017 21:15
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

2 participants