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

Use PipRequirement in bears #1233

Merged
merged 2 commits into from
Jan 1, 2017
Merged

Use PipRequirement in bears #1233

merged 2 commits into from
Jan 1, 2017

Conversation

jayvdb
Copy link
Member

@jayvdb jayvdb commented Jan 1, 2017

Closes #1000

jayvdb added 2 commits January 1, 2017 10:29
Use nbformat~=4.1 instead of nbformat>=4.1
to be consistent with other entries in requirements.txt

Closes coala#1212
Remove all bear requirements from requirements.txt,
and use .ci/generate_bear_requirements.py to create
and verify that the bear-requirements is identical to
the PipRequirement found in the bears.

Update the PipRequirement in the bears to match the
versions currently being installed, and also remove
the PipRequirement for libclang-py, as this
dependency is provided by coala.

Updates setup.py to add an extra 'alldeps' that is be used
by CI to transition to explicitly requesting additional
dependencies so 100% coverage and `coala --ci` work correctly.

Uses pip to install wheels, as it supports extras while
setup.py would require a non-standardised command line option
to be added.

Closes coala#1000
@gitmate-bot
Copy link
Collaborator

Thanks for your contribution!

Reviewing pull requests take really a lot of time and we're all volunteers. Please make sure you go through the following check list and complete them all before pinging someone for a review.

As you learn things over your Pull Request please help others on the chat and on PRs to get their stuff right as well!

@yukiisbored
Copy link
Member

ack jayvdb@2b52dfe

@yukiisbored
Copy link
Member

ack jayvdb@27a06d1

@jayvdb
Copy link
Member Author

jayvdb commented Jan 1, 2017

@rultor merge

@rultor
Copy link

rultor commented Jan 1, 2017

@rultor merge

@jayvdb OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 27a06d1 into coala:master Jan 1, 2017
@rultor
Copy link

rultor commented Jan 1, 2017

@rultor merge

@jayvdb Done! FYI, the full log is here (took me 1min)

@jayvdb jayvdb deleted the bear-dependencies branch January 1, 2017 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants