CI - Remove Unit Tests From Check

stacks-network / stacks-core

The Stacks blockchain implementation

https://docs.stacks.co

GNU General Public License v3.0

3.01k stars 671 forks source link

CI - Remove Unit Tests From Check #5058

Closed BowTiedDevOps closed 3 months ago

BowTiedDevOps commented 3 months ago

When Unit Tests CI Job has a failure, it isn't picked up by the check-tests job (example) due to the continue-on-error: true line in the matrix (here), which causes the job status of the unit tests to show up as success.

This PR removes the checking of the unit-tests job inside check-tests, so this 'failed but showing up as successful' scenario is not happening anymore.

blockstack-devops commented 2 weeks ago

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.