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

Add basic CI job that runs CMake builds #259

Merged
merged 3 commits into from
Jul 18, 2024
Merged

Conversation

chiphogg
Copy link
Contributor

@chiphogg chiphogg commented Jul 10, 2024

For now, we simply run "the default CMake generator on the latest
Ubuntu". Later on, if we tighten up our notion of "supported CMake
configurations", we can add more options here.

Helps #215.

For now, we simply run "the default CMake generator on the latest
Ubuntu".  Later on, if we tighten up our notion of "supported CMake
configurations", we can add more options here.
@chiphogg chiphogg requested a review from a team as a code owner July 10, 2024 02:12
@chiphogg chiphogg requested a review from timhirsh July 10, 2024 02:14
@chiphogg
Copy link
Contributor Author

FYI @timhirsh, we'll need a security review on jwlawson/actions-setup-cmake@802fa1a. Assuming it passes, we can add it to the allowlist in the repo settings, and after that, the job should run remotely. (I already ran it locally using nektos/act, so I know it works.)

@chiphogg
Copy link
Contributor Author

Alternatively, perhaps we should consider batching the security reviews (see #260). Our version of checkout, for example, is pretty old.

Copy link
Member

@timhirsh timhirsh left a comment

Choose a reason for hiding this comment

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

This LGTM, pending completion of the security review 👍

.github/workflows/cmake-build-and-test.yml Outdated Show resolved Hide resolved
Co-authored-by: Tim Hirsh <timhirsh@users.noreply.github.com>
@chiphogg chiphogg added the release notes: ⚙️ repo PR affecting the way the repository works label Jul 10, 2024
@chiphogg chiphogg merged commit fc73b85 into main Jul 18, 2024
11 checks passed
@chiphogg chiphogg deleted the chiphogg/cmake-ci#215 branch July 18, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: ⚙️ repo PR affecting the way the repository works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants