Skip to content

chore: update cucumber messages and gherkin versions - #376

Merged
Daan Timmer (daantimmer) merged 1 commit into
mainfrom
feature/update-cucumber-dependencies
Jul 23, 2026
Merged

Daan Timmer (daantimmer) merged 1 commit into
mainfrom
feature/update-cucumber-dependencies

Conversation

@daantimmer

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings July 23, 2026 11:30
@daantimmer
Daan Timmer (daantimmer) requested review from a team as code owners July 23, 2026 11:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the pinned revisions of the third-party Cucumber dependencies (messages + gherkin) used via CMake FetchContent, aligning the runner with newer upstream releases.

Changes:

  • Bump cucumber/messages FetchContent GIT_TAG to v34.2.0 (commit 073f951...).
  • Bump cucumber/gherkin FetchContent GIT_TAG to v42.0.0 (commit f2f7b51...).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
external/cucumber/messages/CMakeLists.txt Updates the pinned cucumber/messages revision used during the build.
external/cucumber/gherkin/CMakeLists.txt Updates the pinned cucumber/gherkin revision used during the build.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

Copy link
Copy Markdown

✅⚠️MegaLinter analysis: Success with warnings

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 6 0 0 0.32s
✅ ACTION zizmor 6 0 0 0 2.86s
✅ CPP clang-format 239 1 0 0 1.72s
✅ DOCKERFILE hadolint 1 0 0 0.04s
⚠️ MARKDOWN markdownlint 7 4 20 0 0.85s
✅ MARKDOWN markdown-table-formatter 7 4 0 0 0.22s
✅ REPOSITORY betterleaks yes no no 0.88s
✅ REPOSITORY git_diff yes no no 0.03s
✅ REPOSITORY grype yes no no 56.67s
✅ REPOSITORY ls-lint yes no no 0.01s
✅ REPOSITORY secretlint yes no no 4.38s
✅ REPOSITORY syft yes no no 1.91s
✅ REPOSITORY trivy yes no no 12.11s
✅ REPOSITORY trivy-sbom yes no no 0.45s
✅ REPOSITORY trufflehog yes no no 5.51s
⚠️ SPELL lychee 86 1 0 13.62s
✅ YAML prettier 10 0 0 0 0.6s
✅ YAML v8r 10 0 0 9.55s
✅ YAML yamllint 10 0 0 0.58s

Detailed Issues

⚠️ SPELL / lychee - 1 error
📝 Summary
---------------------
🔍 Total..........178
🔗 Unique.........172
✅ Successful.....177
⏳ Timeouts.........0
🔀 Redirected......81
👻 Excluded.........0
❓ Unknown..........0
🚫 Errors...........1
⛔ Unsupported......1

Errors in CONTRIBUTING.md
[404] https://github.com/yourname/amp-cucumber-cpp-runner.git (at 41:36) | Rejected status code: 404 Not Found | Followed 1 redirect. Redirects: https://github.com/yourname/amp-cucumber-cpp-runner.git --[301]--> https://github.com/yourname/amp-cucumber-cpp-runner

Hint: Followed 81 redirects. You might want to consider replacing redirecting URLs with the resolved URLs. Use verbose mode (`-v`/`-vv`) to see redirection details.
Hint: You can configure accepted/rejected response codes with `-a` or `--accept`
⚠️ MARKDOWN / markdownlint - 20 errors
CHANGELOG.md:31 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:38 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Bug Fixes"]
CHANGELOG.md:44 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Chores"]
CHANGELOG.md:51 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:58 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Chores"]
CHANGELOG.md:66 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "⚠ BREAKING CHANGES"]
CHANGELOG.md:70 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:79 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:85 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Chores"]
CHANGELOG.md:93 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "⚠ BREAKING CHANGES"]
CHANGELOG.md:97 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:102 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Bug Fixes"]
CHANGELOG.md:111 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:122 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Chores"]
CHANGELOG.md:130 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "⚠ BREAKING CHANGES"]
CHANGELOG.md:134 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:159 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Bug Fixes"]
CHANGELOG.md:166 error MD024/no-duplicate-heading Multiple headings with the same content [Context: "Chores"]
cmake/WindowsToolchainFilesProvidedBy.md:1 error MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading [Context: "<https://github.com/MarkSchofi..."]
README.md:132 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@9.6.0 --custom-flavor-setup --custom-flavor-linters ACTION_ACTIONLINT,ACTION_ZIZMOR,CPP_CLANG_FORMAT,DOCKERFILE_HADOLINT,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_GIT_DIFF,REPOSITORY_BETTERLEAKS,REPOSITORY_GRYPE,REPOSITORY_LS_LINT,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,SPELL_LYCHEE,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@github-actions

Copy link
Copy Markdown

Test Results

31 tests  ±0   31 ✅ ±0   11s ⏱️ -5s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit bb2cc8e. ± Comparison against base commit fae5994.

@sonarqubecloud

Copy link
Copy Markdown

@daantimmer
Daan Timmer (daantimmer) merged commit 0d8152f into main Jul 23, 2026
24 checks passed
@daantimmer
Daan Timmer (daantimmer) deleted the feature/update-cucumber-dependencies branch July 23, 2026 11:44
Daan Timmer (daantimmer) pushed a commit that referenced this pull request Sep 29, 2026
🤖 I have created a release *beep* *boop*
---


##
[4.2.0](v4.1.0...v4.2.0)
(2026-09-01)


### Features

* Add plugin support
([#378](#378))
([71acefc](71acefc))
* Expose scenario information in scenario and step hooks
([#369](#369))
([fae5994](fae5994))
* Use cucumber/tag-expressions
([#385](#385))
([c4c1bd5](c4c1bd5))


### Bug Fixes

* Add early parse-error reporting during feature collection
([#369](#369))
([fae5994](fae5994))
* Catch exceptions while constructing a step fixture
([#364](#364))
([ca918d5](ca918d5))


### Chores

* Update cucumber messages and gherkin versions
([#376](#376))
([0d8152f](0d8152f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: philips-software-forest-releaser[bot] <80338643+philips-software-forest-releaser[bot]@users.noreply.github.com>
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