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

GitHub Action improvements, including coverage tests! #515

Merged
merged 8 commits into from
May 12, 2024

Conversation

rspier
Copy link
Collaborator

@rspier rspier commented May 12, 2024

Various cleanups of obsolete things from the workflow.
Caching so we don't have to install modules each time (saves 2 minutes)
Run Devel::Cover coverage tests and push results to Coveralls if the want-coverage label is applied.
Explicitly use MariaDB, instead of whatever Debian's default is.

Same as #508, but going to merge without rebase.

rspier added 8 commits May 11, 2024 19:05
…ymore.

PAUSE no longer runs on v5.16, so we can use modern versions.
While they're written in perl, it's clearly the PAUSE test suite.
This shaves off 2 minutes of runtime.

The cache key is based on the perl version and a hash of Makefile.PL, so
adding new modules will trigger a flush.
The tests are no longer hanging, this doesn't seem to be a problem anymore.
@rspier rspier merged commit 62734c1 into andk:master May 12, 2024
3 checks passed
@rspier
Copy link
Collaborator Author

rspier commented May 12, 2024

This is identical to #508, which was approved by @rjbs, but I merged that with rebase, and this was big enough that it should have a merge commit to track it. So I did some things I don't want to mention to make that happen. Hopefully everyone else was asleep and didn't notice. 😉

@rspier rspier deleted the coverage2 branch May 12, 2024 04:35
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