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

Add simple GithubAction for Tox #9

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ChristianKuehnel
Copy link

@ChristianKuehnel ChristianKuehnel commented Jan 20, 2021

The low-hanging fruit for automatic checks is probably getting tox executed on Github Actions.

I was able to run flake8 , mypy and docs successfully.

However I failed with the test suite. We probably need to set up some more things in the environment...

The output of the Github Actions is visible here: https://github.com/ChristianKuehnel/llvm-lnt/actions

@cmatthew
Copy link
Contributor

We do have CI for that already:

http://green.lab.llvm.org/green/job/LNT_Tests/

@ChristianKuehnel
Copy link
Author

Oh sorry I did not see these. I just saw that you have open pull requests and they are not checked via any CI. So I thought it would be nice to have automatic CI coverage for these. Setting up Github Actions is quite easy to do, so I just gave it a shot.

Looking closer the open Pull Requests are also just automatic PRs for security fixes, so no user interaction there I guess...

But you could of course also configure your existing Jenkins server to 1) annotate the commits on GitHub with the build results and 2) to also check pull requests.

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