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

Only print coverage report to console on success. #343

Merged
merged 1 commit into from
Aug 27, 2015
Merged

Only print coverage report to console on success. #343

merged 1 commit into from
Aug 27, 2015

Conversation

kormat
Copy link
Contributor

@kormat kormat commented Aug 26, 2015

This makes it much easier to see errors when they happen.

Also:

  

This makes it much easier to see errors when they happen.

Also:
- Make ./scion.sh test and ./scion.sh coverage work with test filenames
  directly. E.g., this now works:
    ./scion.sh coverage test/lib_zookeeper_test.py
- Cleaned up .gitignore a bit, making patterns anchored at the root of
  the repository when applicable (as this was hiding some issues).
@kormat
Copy link
Contributor Author

kormat commented Aug 27, 2015

Merging, @pszalach can review when he's back.

kormat added a commit that referenced this pull request Aug 27, 2015
Only print coverage report to console on success.
@kormat kormat merged commit f954558 into scionproto:master Aug 27, 2015
@kormat kormat deleted the success_coverage branch August 31, 2015 08:11
@pszal
Copy link
Contributor

pszal commented Sep 7, 2015

lgtm

FR4NK-W pushed a commit to FR4NK-W/osourced-scion that referenced this pull request Feb 14, 2017
Only print coverage report to console on success.
sgmonroy pushed a commit to sgmonroy/scion that referenced this pull request Feb 5, 2019
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.

2 participants