Skip to content

Use consistent multipart limit naming. - #6

Merged
ioquatix merged 1 commit into
mainfrom
parser-limit-naming
Aug 4, 2026
Merged

Use consistent multipart limit naming.#6
ioquatix merged 1 commit into
mainfrom
parser-limit-naming

Conversation

@samuel-williams-shopify

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

Copy link
Copy Markdown
Contributor

Summary

  • rename multipart structure constraints to use the measured quantity followed by _limit
  • rename form-data field, upload, aggregate, and nesting constraints consistently
  • retain total_size_limit for the aggregate across individually limited fields and uploads
  • align constants, keywords, internal state, tests, documentation, and release notes

Depends on socketry/protocol-url#4 being merged and released. The packaged gem dependency remains unchanged.

Testing

Tested locally against the sibling protocol-url PR branch:

  • bundle exec bake test
  • bundle exec rubocop
  • COVERAGE=PartialSummary bundle exec bake decode:index:coverage lib

@ioquatix
ioquatix merged commit 2c98fbb into main Aug 4, 2026
58 of 80 checks passed
@ioquatix
ioquatix deleted the parser-limit-naming branch August 4, 2026 04:54
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.

2 participants