-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add travis ci #4
Conversation
gilgamezh
commented
Nov 8, 2017
- travis CI
- coveralls.io
Changes Unknown when pulling 8abc822 on add_travis_ci into ** on master**. |
Changes Unknown when pulling 3d20e9b on add_travis_ci into ** on master**. |
.travis.yml
Outdated
- "3.6-dev" | ||
- "nightly" # currently points to 3.7-dev | ||
install: | ||
- "pip install -r requirements.txt" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bear in mind that pip install -e .[dev]
already does this. I prefer the shorter method but won't argue much about it, however we should probably be consistent with CONTRIBUTING.md
and avoid duplicating the places where we specify the dependencies.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch! I'm removing it.
And what about the requirements-test.txt?
Changes Unknown when pulling 13905ac on add_travis_ci into ** on master**. |
That's the `[dev]` part, which is in `extra_requires` in `setup.py`
…On Nov 8, 2017 21:18, "Nicolás Demarchi" ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In .travis.yml
<#4 (comment)>:
> @@ -0,0 +1,26 @@
+language: python
+
+python:
+ - "3.4"
+ - "3.5"
+ - "3.5-dev"
+ - "3.6"
+ - "3.6-dev"
+ - "nightly" # currently points to 3.7-dev
+install:
+ - "pip install -r requirements.txt"
good catch! I'm removing it.
And what about the requirements-test.txt?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#4 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AATUZQvuhTuku6ZaKKofNvKMSPHEpBIdks5s0gyWgaJpZM4QW7mv>
.
|
@gilgamezh is the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
@matiasg where are you seeing that? Here all the status checks passed. |
Changes Unknown when pulling 425aea3 on add_travis_ci into ** on master**. |
@Juanlu001 , I was looking at the first commit in the PR. My bad, had to look at the last one. |
Sorry, wrong button :) |
# This is the 1st commit message: pin coveralls version # This is the commit message #2: rever change of pytest-coverage. Pin python-coverage # This is the commit message #3: just as a test, remove coveralls # This is the commit message #4: remove also cov from script 🤦 # This is the commit message #5: pin pytest version # This is the commit message #6: pin hypothesis version # This is the commit message #7: pin hypothesis to 3.55.1 now # This is the commit message #8: do not install pytest-coverage # This is the commit message #9: pin pytest-coverage # This is the commit message #10: install pinned coverage, do not install pytest-coverage :'( # This is the commit message #11: pin attr # This is the commit message #12: attr is attrs # This is the commit message #13: travis, please tell me attr version # This is the commit message #14: go back to original setup.py # This is the commit message #15: exclude coveralls from 3.4 & 3.5. Try 1 # This is the commit message #16: use [] instead of test # This is the commit message #17: install pytest 5.4.3 # This is the commit message #18: travis tell me something # This is the commit message #19: please do tell # This is the commit message #20: ooohhh? # This is the commit message #21: oh 2 # This is the commit message #22: oh 3 # This is the commit message #23: oh 4 # This is the commit message #24: oh 5