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

Github action for running test.sh #1

Closed
wants to merge 2 commits into from
Closed

Conversation

chmeliik
Copy link
Owner

@chmeliik chmeliik commented Nov 13, 2019

Run tests using github actions instead of Travis.

Unfortunately, I was unable to get Coveralls to work:

  • The github action requires that the coverage file be in lcov format, which pytest seems unable to generate.
  • The python coveralls package is unreasonably difficult to use outside of Travis. It works for the basic use case of uploading coverage after each test, but getting Coveralls to comment on the PR or merge the results of parallel tests was beyond my abilities.
  • See also Using for Python code and coverage.py coverallsapp/github-action#4

By the way, if the github editor tells you that Matrix options must only contain primitive values, do not trust it.

@chmeliik chmeliik force-pushed the github-actions branch 14 times, most recently from 2081eb2 to 0575377 Compare November 15, 2019 09:42
Signed-off-by: Adam Cmiel <acmiel@redhat.com>
Do not use the -t flag for docker exec, because the machine running the
action somehow does not count as a TTY.

Force colored output from pytest using --color=yes.

Signed-off-by: Adam Cmiel <acmiel@redhat.com>
@chmeliik chmeliik changed the title Test Github action for running test.sh Nov 15, 2019
@chmeliik chmeliik changed the title Github action for running test.sh [WIP] Github action for running test.sh Nov 15, 2019
@chmeliik chmeliik changed the title [WIP] Github action for running test.sh Github action for running test.sh Nov 15, 2019
@chmeliik chmeliik closed this Nov 6, 2023
@chmeliik chmeliik deleted the github-actions branch November 6, 2023 09:09
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.

1 participant