Skip to content

cpp: Add C++ implementation - #201

Merged
mpkorstanje merged 78 commits into
mainfrom
feature/cpp
Oct 2, 2026
Merged

mpkorstanje merged 78 commits into
mainfrom
feature/cpp

Conversation

@daantimmer

@daantimmer daantimmer commented Aug 7, 2026 •

Copy link
Copy Markdown
Contributor

🤔 What's changed?

  • Added C++ implementation for Query

⚡️ What's your motivation?

  • There was none

🏷️ What kind of change is this?

  • ⚡ New feature (non-breaking change which adds new behaviour)

📋 Checklist:

  • I agree to respect and uphold the Cucumber Community Code of Conduct
  • I've changed the behaviour of the code
    • I have added/updated tests to cover my changes.
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
  • Users should know about my change
    • I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

daantimmer and others added 30 commits July 19, 2026 19:03
…Steps, findAllTestStepStarted, findAllUndefinedParameterTypes, findAttachmentsBy
…edBy, FindTestRunHookStartedBy, FindTestRunStarted
@daantimmer
daantimmer marked this pull request as draft August 31, 2026 20:40
@mpkorstanje

Copy link
Copy Markdown
Member

Okay, no problem.

All the packages that we depend on are available in system packages, so I am updating the test builds to use those available to linux, macos and windows. The downside is we don't have control over the exact installed packages, OTOH, if the tests work with an older version than that should be fine too.

I'd prefer it if we did depend on a specific version. Waiting for system packages is a bit too slow too make a comfortable feedback cycle.

But we should depend on a specific version, not a hash. and Renovate should be able to update that. We've done that before so I hope we can reuse that mechanism.

@daantimmer

Copy link
Copy Markdown
Contributor Author

@mpkorstanje I request your assistance in setting up:

  • renovate
  • VERSION release updates

@daantimmer
daantimmer marked this pull request as ready for review September 18, 2026 10:18
@daantimmer daantimmer changed the title (cpp) Add C++ query implementation [CPP] Initial C++ implementation Sep 23, 2026
@mpkorstanje

mpkorstanje commented Sep 27, 2026 •

Copy link
Copy Markdown
Member

@mpkorstanje I request your assistance in setting up:

* renovate

* VERSION release updates

The VERSION file can be updated by polyglot-release. So that happens automatically as part of the release process.

What I'm missing though for Renovate is a specific file where the dependencies are declared. We had something like this in messages (https://github.com/cucumber/messages/blob/90a083420143561f4fab39528519ee858d1b35a1/cpp/cmake/dependencies.cpm.cmake).

The closest thing that resembles this would be https://github.com/cucumber/query/blob/feature%2Fcpp/cpp/cmake/cucumber_query-config.cmake.in, but I don't see any hashes or version numbers there. I also see a find_package and find_dependency. So to me it is not immediately where and how you fetch the dependencies.

@daantimmer daantimmer changed the title [CPP] Initial C++ implementation [cpp] add C++ implementation Sep 30, 2026
@daantimmer daantimmer changed the title [cpp] add C++ implementation [cpp]: add C++ implementation Sep 30, 2026
@mpkorstanje mpkorstanje changed the title [cpp]: add C++ implementation cpp: add C++ implementation Oct 2, 2026
@mpkorstanje mpkorstanje changed the title cpp: add C++ implementation cpp: Add C++ implementation Oct 2, 2026
@mpkorstanje
mpkorstanje merged commit e327733 into main Oct 2, 2026
33 checks passed
@mpkorstanje
mpkorstanje deleted the feature/cpp branch October 2, 2026 10:17
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