Skip to content

Conversation

@cpennington
Copy link
Contributor

…t not such a performance penalty

.travis.yml Outdated
- python runtests.py

after_success:
- bash <(curl -s https://codecov.io/bash)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this line need to be removed too?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup. I'll remove it (I can edit PRs).

@gvanrossum gvanrossum merged commit 6485ef5 into python:master Oct 25, 2016
@cpennington
Copy link
Contributor Author

Yup. I realized it as soon as I walked out of the office.

On Tue, Oct 25, 2016, 5:07 PM David Fisher [email protected] wrote:

@ddfisher commented on this pull request.

In .travis.yml
#2328 (review):

@@ -13,9 +13,7 @@ install:

  • python setup.py install

script:

    • coverage erase
    • python runtests.py --coverage
    • coverage combine
    • python runtests.py

    after_success:

Does this line need to be removed too?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#2328 (review), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAB2HM4LP1wiCPWF-h8CxKFcGU-JTmwWks5q3m-BgaJpZM4Kgd3V
.

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.

3 participants