Skip to content

refactor!: remove FSTDEP025 - #6909

Merged
jean-michelet merged 2 commits into
nextfrom
refactor/remove-fstdep025
Aug 9, 2026
Merged

jean-michelet merged 2 commits into
nextfrom
refactor/remove-fstdep025

Conversation

@jean-michelet

Copy link
Copy Markdown
Member

Remove FSTDEP025 and throw FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED.

@github-actions github-actions Bot added documentation Improvements or additions to documentation typescript TypeScript related labels Aug 6, 2026
@jean-michelet jean-michelet mentioned this pull request Aug 6, 2026
9 of 12 tasks
@jean-michelet jean-michelet changed the title refactor!: enforce explicit HTTP method overrides refactor!: remove FSTDEP025 Aug 6, 2026
@Tony133 Tony133 added this to the v6.0.0 milestone Aug 6, 2026
@jean-michelet

jean-michelet commented Aug 7, 2026 •

Copy link
Copy Markdown
Member Author

EDIT: done

Merge this before accepting this one:
#6912

@jean-michelet
jean-michelet force-pushed the refactor/remove-fstdep025 branch from e57fa1b to 3eb35b6 Compare August 9, 2026 20:46
Remove FSTDEP025 and throw FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED when addHttpMethod attempts to override a supported method without overrideExisting.

BREAKING CHANGE: addHttpMethod now requires overrideExisting: true when overriding an existing HTTP method.
@jean-michelet
jean-michelet force-pushed the refactor/remove-fstdep025 branch from 3eb35b6 to a6552fe Compare August 9, 2026 20:51
@jean-michelet
jean-michelet requested a review from a team August 9, 2026 21:08
Comment thread docs/Reference/Errors.md Outdated
Co-authored-by: KaKa <[email protected]>
Signed-off-by: Jean Michelet <[email protected]>
@jean-michelet
jean-michelet merged commit bd3e246 into next Aug 9, 2026
30 checks passed
@jean-michelet
jean-michelet deleted the refactor/remove-fstdep025 branch August 9, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation typescript TypeScript related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants