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

ENH(TST): Disable coverage reporting on travis while running pytest #6898

Merged
merged 1 commit into from
Aug 3, 2022

Conversation

yarikoptic
Copy link
Member

I found those reports spit out at the end of the pytest very annoying to avoid
while searching for failed tests. And they aren't really useful since we

  • at large anyways relying on codecov reporting while combining across
    different CIs and runs

  • have report at the end upon success when submitting to codecov. So we
    can look at those if desired, and they are nicly folded in their own step

If desired -- can do for other runs (appveyor etc).

I found those reports spit out at the end of the pytest very annoying to avoid
while searching for failed tests.  And they aren't really useful since we

- at large anyways relying on codecov reporting while combining across
different CIs and runs

- have `report` at the end upon success when submitting to codecov. So we
can look at those if desired, and they are nicly folded in their own step

If desired -- can do for other runs (appveyor etc).
@yarikoptic yarikoptic added DX developer experience semver-internal Changes only affect the internal API labels Jul 29, 2022
@yarikoptic
Copy link
Member Author

ok, no objections. CI fails known/unrelated, I like it, let's proceed.

@yarikoptic yarikoptic merged commit f4726d8 into datalad:maint Aug 3, 2022
@yarikoptic yarikoptic deleted the rf-no-covreport branch October 14, 2022 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX developer experience semver-internal Changes only affect the internal API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant