Skip to content

Use retainLines option when generating code#182

Merged
davidjgoss merged 2 commits intomainfrom
retain-lines
Jan 15, 2026
Merged

Use retainLines option when generating code#182
davidjgoss merged 2 commits intomainfrom
retain-lines

Conversation

@davidjgoss
Copy link
Collaborator

@davidjgoss davidjgoss commented Jan 15, 2026

🤔 What's changed?

Use babel's retainLines option when generating code.

⚡️ What's your motivation?

To have tests correctly source mapped in the spec output, we require --enable-source-maps. If that's missed by the user, this option will at least ensure the line is correct even though the column won't make sense.

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

@davidjgoss davidjgoss marked this pull request as ready for review January 15, 2026 12:47
@davidjgoss davidjgoss merged commit b43c551 into main Jan 15, 2026
11 checks passed
@davidjgoss davidjgoss deleted the retain-lines branch January 15, 2026 12:47
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