Skip to content

Introduce a limit error. - #8

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
introduce-limit-error
Aug 5, 2026
Merged

Introduce a limit error.#8
samuel-williams-shopify merged 1 commit into
mainfrom
introduce-limit-error

Conversation

@samuel-williams-shopify

@samuel-williams-shopify samuel-williams-shopify commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add Protocol::Multipart::LimitError
  • use it for multipart structure and form-data byte limits
  • leave nesting-limit errors owned by the supplied result object

Tests

  • bundle exec bake test with protocol-url 0.9.0
  • bundle exec bake decode:index:coverage lib
  • bundle exec rubocop

@samuel-williams-shopify
samuel-williams-shopify marked this pull request as draft August 5, 2026 03:38
@samuel-williams-shopify
samuel-williams-shopify marked this pull request as ready for review August 5, 2026 03:56
@samuel-williams-shopify
samuel-williams-shopify merged commit 5e3bb9b into main Aug 5, 2026
40 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the introduce-limit-error branch August 5, 2026 04:02
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