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

Split test requirements out and add constraint #81

Merged
merged 3 commits into from
Oct 1, 2018

Conversation

tofu-rocketry
Copy link
Member

The weekly cron build failed which raised an issue with a dependency of a dependency not supporting Python 2.6 any more. This PR fixes that (though the longer term fix is to deprecate Python 2.6 support).

  • Move test requirements into separate requirements file to make it
    clearer that they are only needed for running unit tests.
  • Add constraints file called from the test requirements file to limit
    one of coveralls' dependencies to an earlier version that works with
    Python 2.6.

- Move test requirements into seperate requirements file to make it
clearer that they are only needed for running unit tests.
- Add constraints file called from the test requirements file to limit
one of coveralls' dependencies to an ealier version that works with
Python 2.6.
gregcorbett
gregcorbett previously approved these changes Sep 26, 2018
@gregcorbett
Copy link
Member

I am happy to approve the changes, do you want to squash the commits down before I review though?

@gregcorbett gregcorbett self-requested a review October 1, 2018 15:14
@tofu-rocketry tofu-rocketry merged commit 5bbbbdf into apel:dev Oct 1, 2018
@tofu-rocketry tofu-rocketry deleted the requirements-fix branch October 1, 2018 15:16
@tofu-rocketry tofu-rocketry modified the milestones: 2.3.1, 2.4.0 Oct 15, 2018
@tofu-rocketry tofu-rocketry modified the milestones: 2.4.0, 2.3.1-2 Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants