Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

chore: fix CI pipeline #31

Merged
merged 2 commits into from
Jul 19, 2024
Merged

Conversation

jedel1043
Copy link
Contributor

Description

Fixes our CI so that we're testing PRs and pushes to main.

How was the code tested?

Locally in my Ubuntu Jammy development setup.

Checklist

  • I am the author of these changes, or I have the rights to submit them.
  • I have added the relevant changes to the README and/or documentation.
  • I have self reviewed my own code.
  • All requested changes and/or review comments have been resolved.

@jedel1043
Copy link
Contributor Author

Blocked until charmed-hpc/slurmrestd-operator#20 gets merged.

@NucciTheBoss
Copy link
Member

@jedel1043 I have reviewed your pull request against slurmrestd 🕵️

@NucciTheBoss
Copy link
Member

Unblocked since I merged the CI pull request against slurmrestd last night. Nice to have the pipelines up and running again!

@jedel1043 jedel1043 marked this pull request as ready for review July 19, 2024 15:32
Copy link
Member

@NucciTheBoss NucciTheBoss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you for this work 🚀

@NucciTheBoss NucciTheBoss merged commit 5f78fa7 into charmed-hpc:main Jul 19, 2024
6 checks passed
@jedel1043 jedel1043 deleted the fix-ci-pipeline branch July 19, 2024 17:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants