Skip to content

Conversation

mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Sep 5, 2025

⚡️ What's your motivation?

Only unambiguous steps, i.e. steps with a single list of matches should be included in the report.

🏷️ What kind of change is this?

  • 🐛 Bug fix (non-breaking change which fixes a defect)

📋 Checklist:

  • Add sample from Cucumber-JVM 7.26.0 with ambiguous steps with arguments
  • Add sample for ambiguous steps compatibility-kit#156 for verification
  • 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.

@mpkorstanje mpkorstanje force-pushed the fix-step-match-arguments branch from b44acac to 446543c Compare September 9, 2025 15:15
@mpkorstanje mpkorstanje changed the title Only include match arguments highlighting for ambiguous steps Only include match arguments for ambiguous steps Sep 9, 2025
@mpkorstanje mpkorstanje changed the title Only include match arguments for ambiguous steps Only include match arguments for unambiguous steps Sep 9, 2025
@mpkorstanje mpkorstanje force-pushed the fix-step-match-arguments branch from dfda829 to 29fe08d Compare September 9, 2025 15:37
@mpkorstanje mpkorstanje marked this pull request as ready for review September 9, 2025 15:37
@mpkorstanje mpkorstanje merged commit eabf944 into main Sep 9, 2025
5 checks passed
@mpkorstanje mpkorstanje deleted the fix-step-match-arguments branch September 9, 2025 16:03
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.

1 participant