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

ci: use reuseable workflows #255

Merged
merged 3 commits into from
Dec 23, 2023
Merged

Conversation

huitseeker
Copy link
Member

No description provided.

@huitseeker huitseeker changed the title ci: use resuable workflows ci: use reusable workflows Dec 21, 2023
@huitseeker huitseeker changed the title ci: use reusable workflows ci: use reuseable workflows Dec 21, 2023
@huitseeker
Copy link
Member Author

huitseeker commented Dec 21, 2023

Hmm, I forgot to remove the part of the CI workflow that runs the doctests, which makes the failure of https://github.com/lurk-lab/neptune/actions/runs/7294189991/job/19878612122?pr=255 weird.

@huitseeker
Copy link
Member Author

This is the line that makes this tricky, reduced the PR to something simpler for now:
https://github.com/lurk-lab/neptune/blob/main/gbench/Cargo.toml#L25

- uses: actions/checkout@v4
with:
repository: lurk-lab/ci-workflows
- uses: ./.github/actions/ci-env
Copy link
Member

Choose a reason for hiding this comment

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

We can also use the gpu-setup action to cut down on more lines in both the cuda and opencl jobs

Copy link
Member Author

Choose a reason for hiding this comment

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

That's definitely the next step!

.github/workflows/rust.yml Outdated Show resolved Hide resolved
@huitseeker
Copy link
Member Author

huitseeker commented Dec 22, 2023

Pausing work on this: we need to switch to https://github.com/dtolnay/rust-toolchain first, before we use our reusable workflows.
Edit: the coast is clear!

@huitseeker huitseeker force-pushed the reusable-ci branch 2 times, most recently from 6bbb7f7 to ca5617a Compare December 23, 2023 16:47
@huitseeker huitseeker added this pull request to the merge queue Dec 23, 2023
Merged via the queue into argumentcomputer:main with commit 1ac158e Dec 23, 2023
7 checks passed
@huitseeker huitseeker deleted the reusable-ci branch December 23, 2023 21:03
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