Skip to content

Conversation

davidjgoss
Copy link
Contributor

@davidjgoss davidjgoss commented Sep 22, 2025

🤔 What's changed?

Support a user-provided name for test run hooks and include it on the corresponding Hook messages.

⚡️ What's your motivation?

Allow users to name hooks to better understand what's going on. Also, meet the messages contract more fully.

🏷️ 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.

This text was originally generated from a template, then edited by hand. You can modify the template here.

@coveralls
Copy link

Coverage Status

coverage: 97.79% (+0.002%) from 97.788%
when pulling 6f0ba91 on named-test-run-hooks
into 63cc0ce on main.

@davidjgoss davidjgoss marked this pull request as ready for review September 22, 2025 23:40
@davidjgoss davidjgoss merged commit bb088ff into main Sep 22, 2025
8 checks passed
@davidjgoss davidjgoss deleted the named-test-run-hooks branch September 22, 2025 23:40
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.

2 participants