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.

Sign up for GitHub

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

Jump to bottom

CI: Rework workflows to make them a bit easier to maintain #360

Merged
merged 7 commits into from
Sep 9, 2024
Merged

CI: Rework workflows to make them a bit easier to maintain #360

merged 7 commits into from
Sep 9, 2024

Conversation

tmadlener
Copy link
Contributor

@tmadlener tmadlener commented Sep 9, 2024 •
edited
Loading

BEGINRELEASENOTES

  • CI: Update versions of github actions where appropriate
  • CI: simplify workflows that build podio and EDM4hep by using cmake commands to invoke the build tool
  • CI: Enable more features from podio for building EDM4hep on top of
    • Also fix a buggy printout that states the commit of podio that was used for building

ENDRELEASENOTES

@tmadlener
CI: Simplify workflows that build podio on the fly
a14e467 
- Update version actions
- Use checkout action to get podio
- Use podio setup scripts to set environment
- Enable more podio features
@tmadlener
Use Cmake to invoke the build tool
154a07d
@tmadlener
CI: simplify the pre-commit action
40ca23c 
- Use cmake to invoke build commands to make script a bit shorter
@tmadlener
Make sure to source the setup script
8e74f51
@tmadlener tmadlener changed the title CI: Minor updates and some simplifications CI: Rework workflows to make them a bit easier to maintain Sep 9, 2024
@tmadlener tmadlener merged commit b99ba9f into key4hep:main Sep 9, 2024
7 of 10 checks passed
@tmadlener tmadlener deleted the small-ci-fixes branch September 9, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tmadlener