Skip to content

TODO: Doc update: Update default behaviour of use_for_snippets - #393

Open
luke-hill wants to merge 1 commit into
mainfrom
luke-hill-patch-1
Open

luke-hill wants to merge 1 commit into
mainfrom
luke-hill-patch-1

Conversation

@luke-hill

@luke-hill luke-hill commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Needs consensus across all maintainers. cc/ @davidjgoss @mpkorstanje @daantimmer @gasparnagy and prob others.

🤔 What's changed?

Had a mini discussion. And we feel as though we should probably change use_for_snippets to default from true to false. In some languages this has been "partially" done.

⚡️ What's your motivation?

Avoids bad regex spam. Users can still stipulate these to be used by using the true flag

🏷️ What kind of change is this?

  • 📖 Documentation (improvements without changing code)
  • 🏦 Refactoring/debt/DX (improvement to code design, tooling, etc. without changing 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.

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

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying cucumber-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 334612a
Status: ✅  Deploy successful!
Preview URL: https://db7a589b.cucumber-website.pages.dev
Branch Preview URL: https://luke-hill-patch-1.cucumber-website.pages.dev

View logs

@luke-hill

Copy link
Copy Markdown
Contributor Author

FYI I've not made the doc change yet, just made this PR so we don't forget.

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