Skip to content

lib: drop setDefaultRoute and getDefaultRoute methods#4485

Merged
RafaelGSS merged 1 commit intofastify:nextfrom
RafaelGSS:drop-default-route
Dec 29, 2022
Merged

lib: drop setDefaultRoute and getDefaultRoute methods#4485
RafaelGSS merged 1 commit intofastify:nextfrom
RafaelGSS:drop-default-route

Conversation

@RafaelGSS
Copy link
Member

Signed-off-by: RafaelGSS rafael.nunu@hotmail.com

Following up: #4480

Checklist

Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
@RafaelGSS RafaelGSS added the semver-major Issue or PR that should land as semver major label Dec 27, 2022
@mcollina
Copy link
Member

This can land in v4, can you rebase onto main?

@RafaelGSS
Copy link
Member Author

This can land in v4, can you rebase onto main?

Isn't it a breaking change?

@mcollina
Copy link
Member

ah no, this is the correct one! I mixed the two PRs.

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@RafaelGSS RafaelGSS merged commit b8e201a into fastify:next Dec 29, 2022
@github-actions
Copy link

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 Dec 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

semver-major Issue or PR that should land as semver major

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants