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

Enable running all tests to completion even if one python version failed #295

Closed
wants to merge 1 commit into from

Conversation

macisamuele
Copy link
Contributor

What:
Ensure that we do run all the tests (platforms/python-versions) and that we don't immediately stop if one has failed.
This is a follow-up from #292

Why:
In case of test failures is extremely interesting to have information about "is this failure a genuine bug or is an issue with a single python version?".

How:
Ensure that github workflow is correctly defined. Doc in here

Risks:

n/a

Checklist:

  • Added tests, if you've added code that should be tested
  • Updated the documentation, if you've changed APIs
  • Ensured the test suite passes
  • Made sure your code lints
  • Completed the Contributor License Agreement ("CLA")

@macisamuele macisamuele requested a review from deathowl March 29, 2021 11:28
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Mar 29, 2021
@coveralls
Copy link

Pull Request Test Coverage Report for Build 697672057

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.159%

Totals Coverage Status
Change from base Build 687383861: 0.0%
Covered Lines: 2563
Relevant Lines: 2722

💛 - Coveralls

@facebook-github-bot
Copy link

@macisamuele has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link

@macisamuele merged this pull request in f42e0e9.

@macisamuele macisamuele deleted the run-all-tests branch April 4, 2021 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants