Skip to content

Conversation

@thomasisensee
Copy link
Member

@thomasisensee thomasisensee commented Nov 26, 2025

We can now automatically propagate changes from this repo's main branch to the main branch of the cpp-project-template, as suggested by this issue. The workflow ran successfully (link) and created the commit: ssciwr/cpp-project-template@main...sync-test

The workflow bakes the project using the following configuration (up for discussion):

default_context:
project_name: "Your Project"
remote_url: "None"
project_slug: "your-project"
full_name: "Your Name"
license: "MIT"
use_submodules: "No"
header_only: "No"
external_dependency: "None"
github_acitons_ci: "Yes"
gitlab_ci: "No"
readthedocs: "Yes"
doxygen: "Yes"
cxx_minimum_standard: "20"
python_bindings: "No"
pypi_release: "No"
codecovio: "Yes"

TODO:

  • Possibly insert a header in the README to indicate that the cpp-project-template was generated from this cookiecutter
    EDIT: Added Acknowledgement of cookiecutter at README's end
  • Use the main branches instead of the test branches

on:
push:
branches:
- sync-project-template

Copy link
Member

@lkeegan lkeegan left a comment

Choose a reason for hiding this comment

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

thanks @thomasisensee this looks good to me! definitely an improvement to automatically sync changes from the cookiecutter to this template.

one question about catch2 - do we need to install it in CI if we're using fetchcontent in cmake? (I guess this might be more of a question for the cookiecutter than for your PR though)

Co-authored-by: Liam Keegan <liam@keegan.ch>
@thomasisensee
Copy link
Member Author

Valid point @lkeegan, we should decide on one option and either install Catch2 manually or via FetchContent. But as you said, I'd rather make this change in PR #135.

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.

3 participants