Skip to content

Fix: GHA ci.yml doesn't fail if exercise tests are failing #218

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

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

fapdash
Copy link
Contributor

@fapdash fapdash commented Oct 5, 2022

https://github.com/exercism/emacs-lisp/blob/8848da5d17e639f61bb9840fd55d12ee5b1873f3/.github/workflows/ci.yml
didn't exit with an error code if exercise tests where failing

@fapdash
Copy link
Contributor Author

fapdash commented Oct 5, 2022

Test failure of emacs-lisp/main/ci is expected until #217 is merged.

@ErikSchierboom
Copy link
Member

@fapdash Could you rebase?

-
https://github.com/exercism/emacs-lisp/blob/8848da5d17e639f61bb9840fd55d12ee5b1873f3/.github/workflows/ci.yml
  didn't exit with an error code if exercise tests where failing
- This was noticed because the tests for
  exercism#210 did fail
  but the bash script didn't register the failure.
@fapdash fapdash force-pushed the fix-ci-never-failing branch from 6fa7b64 to e6e1a9d Compare October 5, 2022 15:24
@ErikSchierboom ErikSchierboom reopened this Oct 5, 2022
@ErikSchierboom ErikSchierboom merged commit d3fa4ed into exercism:main Oct 6, 2022
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