Skip to content

Conversation

@kennethrioja
Copy link
Contributor

@kennethrioja kennethrioja commented Jan 12, 2026

Summary of changes

  • test.yml: while testing the current branch, it keeps now the total code coverage in %, as well as code coverage per changed file. If there are modification on .rb files, it also runs the entire test suite in the base branch. Then it compares code coverage difference between the two branches:
    • if there is an improvement or none, it passes
    • if there is a decrease in code coverage %, it fails

Motivation and context

This follows up Finn's comment : #1149 (comment)

testing, I think it would be nice to set-up a GH Action that fails if the code coverage % decreases in a PR.

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree
    to license it to the TeSS codebase under the
    BSD license.

DEPRECATED

Feature: It used to write a message on the PR, see screenshot, but that was when testing on my personal repo, where I had the authority to write, when writing in ElixirTeSS:master, we need to have a secret key/variable (even when permissions pull-request: write)... for now let's keep it simple

Screenshots

image

@kennethrioja kennethrioja force-pushed the code-coverage branch 3 times, most recently from 25907ce to d720705 Compare January 12, 2026 13:18
@kennethrioja
Copy link
Contributor Author

Must rewrite it in another action... wip

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