Skip to content

Conversation

@JeffGarland
Copy link
Member

This will fix godbolt by handling the case where config.hpp doesn't exist

@JeffGarland JeffGarland linked an issue Jul 5, 2025 that may be closed by this pull request
@JeffGarland JeffGarland requested review from camio and foonathan July 5, 2025 16:10
@JeffGarland
Copy link
Member Author

@nickelpro note that again for this repo 3.28 cmake setting doesn't work. There's no modules in this repo yet -- so I reverted to 3.25. so the ci will function correctly. Anyway, the 3.28 update isn't for free yet...

uses: lukka/get-cmake@latest
with:
cmakeVersion: ~3.25.0
cmakeVersion: ~3.28.0
Copy link
Member Author

@JeffGarland JeffGarland Jul 5, 2025

Choose a reason for hiding this comment

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

@nickelpro this change is ok, but had to leave 3.25 in top level cmake

@JeffGarland JeffGarland added this pull request to the merge queue Jul 6, 2025
@JeffGarland JeffGarland removed this pull request from the merge queue due to a manual request Jul 6, 2025
@JeffGarland JeffGarland merged commit 47ce759 into main Jul 6, 2025
18 of 20 checks passed
@JeffGarland JeffGarland deleted the 25-fix-godbolt branch July 6, 2025 05:23
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.

godbolt any_view is broken by generated .hpp include

3 participants