You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Feb 19, 2022. It is now read-only.
The dev branch doesn't look to pass any tests... I mean when we submit a PR it's kinda sad to see "All checks have failed" and it's not even our code's fault ;)
Would be nice to fix this stuff, so when somebody is submitting a PR, it's all green if it's all good, and it's red when a specific test was broken, after all, that's the purpose of unit testing I guess.
If tests always fail, nobody will check them and they are useless. Then they should be temporarily disabled until they are properly fixed.
The text was updated successfully, but these errors were encountered:
firepol
changed the title
Fix travis tests
Fix travis / tests or temporarily disable tests
Apr 30, 2018
Yepp, you're right - they need fixing. I disagree on them being useless, however. Anyone submitting their PRs should have run tests before submitting, making sure their tests pass, and no other tests break.
Don't take me wrong, I don't think that disabling ALL tests is the way to go. But say there are these 2-3 annoying tests that always fail, what's the point to have them fail all the time? Disable them and create an issue (so t won't be forgotten) in github to fix the problem and re-enable them later. So at least all the other ones, the ones that are green, will prevail and the build will be green again. So if a new PR will mess up with one of those tests, the build will be red because of a mistake in the PR, not because these 2-3 tests fail all the time...
Thx for considering this... it makes submitting a PR more fun then seeing "All checks have failed" ... "The Travis CI build failed"
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The dev branch doesn't look to pass any tests... I mean when we submit a PR it's kinda sad to see "All checks have failed" and it's not even our code's fault ;)
Would be nice to fix this stuff, so when somebody is submitting a PR, it's all green if it's all good, and it's red when a specific test was broken, after all, that's the purpose of unit testing I guess.
If tests always fail, nobody will check them and they are useless. Then they should be temporarily disabled until they are properly fixed.
The text was updated successfully, but these errors were encountered: