-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Active Coveralls on Pull Requests #5089
Conversation
OlivierCavadenti
commented
Mar 25, 2022
- Active existing coveralls Github Action on pull requests to have code coverage control (avoid no covered code)
@kibertoad It seems coverall bot dont leave comment on the PR, can you check this ? EDITI : it seems it's not possible from forked repo : coverallsapp/github-action#7 |
@kibertoad I intentionnaly drop the code coverage by skip some tests, but it seems the job is already green. |
Sorry for delay, will take a l9ok tomorrow |
I've set threshold for change to 0.0 |
Coveralls job still green, but it seems an issue : lemurheavy/coveralls-public#1623 |
Can remove threshold by blank the input instead of put 0.0 plz ? @kibertoad |
@OlivierCavadenti done |
It's possible to send me credentials for Coveralls by mail ? I will tests some configuration tomorrow. |
@OlivierCavadenti Try using github account for auth, if you can't see knex repo there, I'll add permissions |
@kibertoad I can connect and I see project but I can't modify anything |
@OlivierCavadenti Can you try now? |
it's ok now thanks ! |
8cbec99
to
e68bb05
Compare
2be230a
to
25879f1
Compare
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... |