Skip to content

Reject extended whitespace in tags#438

Closed
mpkorstanje wants to merge 9 commits intomainfrom
extended-whitespace-in-tags
Closed

Reject extended whitespace in tags#438
mpkorstanje wants to merge 9 commits intomainfrom
extended-whitespace-in-tags

Conversation

@mpkorstanje
Copy link
Copy Markdown
Contributor

🤔 What's changed?

⚡️ What's your motivation?

Validate findings from #436 against other implementations.

🏷️ What kind of change is this?

  • 📖 Documentation (improvements without changing code)
  • 🏦 Refactoring/debt/DX (improvement to code design, tooling, etc. without changing behaviour)
  • 🐛 Bug fix (non-breaking change which fixes a defect)
  • ⚡ New feature (non-breaking change which adds new behaviour)
  • 💥 Breaking change (incompatible changes to the API)

♻️ Anything particular you want feedback on?

📋 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.

@mpkorstanje mpkorstanje force-pushed the extended-whitespace-in-tags branch from cf86b19 to e549f10 Compare July 25, 2025 18:13
@mpkorstanje mpkorstanje force-pushed the extended-whitespace-in-tags branch from 0b05ca4 to 733f828 Compare July 27, 2025 01:28
@mpkorstanje mpkorstanje mentioned this pull request Jul 27, 2025
7 tasks
mpkorstanje added a commit that referenced this pull request Jul 27, 2025
While investigating the impact of #436 and #438 some discrepancies came
to light.

Lines consisting entirely of whitespace are considered empty. As such
the tokens for these lines should have 0 indentation and start at column
1.

This behavior diverged for c, .net, go, java and php. As tokens are not
part of the public API, this change has no impact on the observable
behavior of the parser.
@mpkorstanje mpkorstanje deleted the extended-whitespace-in-tags branch July 27, 2025 15:12
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