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 github actions #379

Merged
merged 2 commits into from Mar 14, 2021
Merged

Add github actions #379

merged 2 commits into from Mar 14, 2021

Conversation

bollwyvl
Copy link
Contributor

@bollwyvl bollwyvl commented Dec 8, 2020

- if: ${{ matrix.os == 'ubuntu' && matrix.python-version == '3.8' }}
run: |
doit coverage
coveralls || echo "TODO: remove when run on a repo that is set up"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What this means? "which kind of repo setup"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hadn't set up codecov in a while, since the bad old days when a token was needed. I assume it's just working now?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok understood. No, it is not working:

INFO:coveralls:422
INFO:coveralls:{"message":"Couldn't find a repository matching this job.","error":true}
TODO: remove when run on a repo that is set up

I dont understand this error message. It seems to work well on travis... Any idea why ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, there's probably a token or something. And looking at it, of course, it's coveralls. It looks it does need a token in secrets, which I don't like to bother with until i absolutely must.

I have been using codecov recently, which doesn't require a secret for public repos, and only generates one PR comment, which it updates, which I find more convenient... will do a follow-up PR with a demo.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR up #384 with lots of ✔️

@schettino72 schettino72 merged commit 0aa4cc5 into pydoit:master Mar 14, 2021
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.

Add GitHub Actions?
2 participants