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

Active Coveralls on Pull Requests #5089

Merged

Conversation

OlivierCavadenti
Copy link
Collaborator

  • Active existing coveralls Github Action on pull requests to have code coverage control (avoid no covered code)

@OlivierCavadenti
Copy link
Collaborator Author

OlivierCavadenti commented Mar 25, 2022

@kibertoad It seems coverall bot dont leave comment on the PR, can you check this ?
https://javascript.plainenglish.io/enable-coveralls-to-leave-comments-on-prs-4c88fbeee363

EDITI : it seems it's not possible from forked repo : coverallsapp/github-action#7

@OlivierCavadenti
Copy link
Collaborator Author

@kibertoad
I made some tests, and it seems at this time it will be difficult to automate posting comments on PR with coveralls (write permissions issue), it's not so important by the way.
One thing I will like by the way is to have a failing job when code coverage drop.

image

I intentionnaly drop the code coverage by skip some tests, but it seems the job is already green.
Can you check in the repo settings of Coveralls if they are any threshold for coverage decrease and remove it ?

@kibertoad
Copy link
Collaborator

Sorry for delay, will take a l9ok tomorrow

@kibertoad
Copy link
Collaborator

I've set threshold for change to 0.0

@OlivierCavadenti
Copy link
Collaborator Author

Coveralls job still green, but it seems an issue : lemurheavy/coveralls-public#1623

@OlivierCavadenti
Copy link
Collaborator Author

Can remove threshold by blank the input instead of put 0.0 plz ? @kibertoad

@kibertoad
Copy link
Collaborator

@OlivierCavadenti done

@kibertoad
Copy link
Collaborator

image

@OlivierCavadenti
Copy link
Collaborator Author

It's possible to send me credentials for Coveralls by mail ? I will tests some configuration tomorrow.

@kibertoad
Copy link
Collaborator

@OlivierCavadenti Try using github account for auth, if you can't see knex repo there, I'll add permissions

@OlivierCavadenti
Copy link
Collaborator Author

@kibertoad I can connect and I see project but I can't modify anything

@kibertoad
Copy link
Collaborator

@OlivierCavadenti Can you try now?

@OlivierCavadenti
Copy link
Collaborator Author

@OlivierCavadenti Can you try now?

it's ok now thanks !

@OlivierCavadenti OlivierCavadenti force-pushed the coveralls-gitactions branch 2 times, most recently from 8cbec99 to e68bb05 Compare March 31, 2022 18:25
@OlivierCavadenti OlivierCavadenti merged commit 43dcc79 into knex:master Apr 9, 2022
@OlivierCavadenti OlivierCavadenti deleted the coveralls-gitactions branch April 9, 2022 12:33
@OlivierCavadenti
Copy link
Collaborator Author

Seems coverage decrease don't work but it seems to be useful to have the coverage on each pull request anyway. I will check later if I finally can fix this...

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.

None yet

2 participants