Skip to content

Conversation

@Joseph-Edwards
Copy link
Collaborator

In #666, we added a custom action in place of https://github.com/gap-actions/run-pkg-tests that treats warnings as errors and makes running Valgrind easier. As of gap-actions/run-pkg-tests#35, this is now integrated into the gap action.

This PR reuses the gap action.

dependabot bot and others added 2 commits April 2, 2025 16:09
Bumps [gap-actions/run-pkg-tests](https://github.com/gap-actions/run-pkg-tests) from 2 to 3.
- [Commits](gap-actions/run-pkg-tests@v2...v3)

---
updated-dependencies:
- dependency-name: gap-actions/run-pkg-tests
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@Joseph-Edwards
Copy link
Collaborator Author

The Valgrind action hasn't been triggered, so I am testing it here:
https://github.com/Joseph-Edwards/Digraphs/actions/runs/14223354028/job/39856753182

@james-d-mitchell
Copy link
Member

Thanks @Joseph-Edwards is this ready? Why did the job not trigger?

@Joseph-Edwards
Copy link
Collaborator Author

Yeah this is ready to go. It seems like the Valgrind job only gets triggered on releases or when the C code gets edited, so it's non-run was to be expected.

@james-d-mitchell james-d-mitchell merged commit 4fdf2f1 into digraphs:stable-1.10 Apr 4, 2025
27 checks passed
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.

2 participants