Skip to content

test: port bodyLimit.test.js#5781

Merged
jsumners merged 4 commits into
fastify:mainfrom
rstagi:test/port-bodyLimit.test.js
Oct 27, 2024
Merged

test: port bodyLimit.test.js#5781
jsumners merged 4 commits into
fastify:mainfrom
rstagi:test/port-bodyLimit.test.js

Conversation

@rstagi

@rstagi rstagi commented Oct 26, 2024

Copy link
Copy Markdown
Contributor

Checklist

@jsumners jsumners left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@Uzlopak Uzlopak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

while we are at it. PLease rename bodyLimit.test.js to body-limit.test.js

Thank you

@rstagi rstagi force-pushed the test/port-bodyLimit.test.js branch from d53fc0b to 93b01c3 Compare October 27, 2024 12:05

@Uzlopak Uzlopak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@Uzlopak Uzlopak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@jsumners jsumners merged commit 1124b47 into fastify:main Oct 27, 2024
@github-actions

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants