Skip to content

Add AttachmentExternalisingStream#89

Merged
davidjgoss merged 3 commits intomainfrom
externalise
Apr 11, 2026
Merged

Add AttachmentExternalisingStream#89
davidjgoss merged 3 commits intomainfrom
externalise

Conversation

@davidjgoss
Copy link
Copy Markdown
Member

@davidjgoss davidjgoss commented Apr 10, 2026

🤔 What's changed?

Adds a new stream class for externalising attachments per some given options.

This can precede CucumberHtmlStream in a pipeline that produces an HTML report, externalising attachments as needed.

⚡️ What's your motivation?

cucumber/cucumber-js#2781

🏷️ 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 April 11, 2026 10:16
@davidjgoss davidjgoss merged commit 0539eef into main Apr 11, 2026
5 checks passed
@davidjgoss davidjgoss deleted the externalise branch April 11, 2026 10:16
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