Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

follow #109 - removed remnants of error handler added by #18 #164

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

lamweili
Copy link
Contributor

#109 dropped the error handler, but there are some remnants of it added by #18 that are not cleaned up.
This PR cleans them up and reverts the changes done by #18.

Checklist

…fy#18

Signed-off-by: Lam Wei Li <lam_wei_li@hotmail.com>
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

@mcollina mcollina merged commit 822ae7a into fastify:master Sep 2, 2024
14 checks passed
github-merge-queue bot pushed a commit to rustymotors/server that referenced this pull request Sep 17, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[@fastify/sensible](https://redirect.github.com/fastify/fastify-sensible)
| [`^5.6.0` ->
`^6.0.0`](https://renovatebot.com/diffs/npm/@fastify%2fsensible/5.6.0/6.0.0)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@fastify%2fsensible/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@fastify%2fsensible/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@fastify%2fsensible/5.6.0/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@fastify%2fsensible/5.6.0/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
|
[@fastify/sensible](https://redirect.github.com/fastify/fastify-sensible)
| [`^5.6.0` ->
`^6.0.0`](https://renovatebot.com/diffs/npm/@fastify%2fsensible/5.5.0/6.0.0)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@fastify%2fsensible/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@fastify%2fsensible/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@fastify%2fsensible/5.5.0/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@fastify%2fsensible/5.5.0/6.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>fastify/fastify-sensible (@&#8203;fastify/sensible)</summary>

###
[`v6.0.0`](https://redirect.github.com/fastify/fastify-sensible/releases/tag/v6.0.0)

[Compare
Source](https://redirect.github.com/fastify/fastify-sensible/compare/v5.6.0...v6.0.0)

#### What's Changed

- perf: replace fast-deep-equal with dequal by
[@&#8203;gurgunday](https://redirect.github.com/gurgunday) in
[fastify/fastify-sensible#160
- Merge `next` into `master` by
[@&#8203;jsumners](https://redirect.github.com/jsumners) in
[fastify/fastify-sensible#161
- build(deps-dev): bump
[@&#8203;types/node](https://redirect.github.com/types/node) from
20.14.13 to 22.0.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot) in
[fastify/fastify-sensible#163
- refactor: rename SensibleOptions into FastifySensibleOptions by
[@&#8203;jean-michelet](https://redirect.github.com/jean-michelet) in
[fastify/fastify-sensible#162
- build(deps): bump fastify/workflows from 4.1.0 to 5.0.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot) in
[fastify/fastify-sensible#166
- chore: update min fastify version by
[@&#8203;Fdawgs](https://redirect.github.com/Fdawgs) in
[fastify/fastify-sensible#165
- follow
[#&#8203;109](https://redirect.github.com/fastify/fastify-sensible/issues/109)
- removed remnants of error handler added by
[#&#8203;18](https://redirect.github.com/fastify/fastify-sensible/issues/18)
by [@&#8203;lamweili](https://redirect.github.com/lamweili) in
[fastify/fastify-sensible#164

#### New Contributors

- [@&#8203;jean-michelet](https://redirect.github.com/jean-michelet)
made their first contribution in
[fastify/fastify-sensible#162
- [@&#8203;lamweili](https://redirect.github.com/lamweili) made their
first contribution in
[fastify/fastify-sensible#164

**Full Changelog**:
fastify/fastify-sensible@v5.6.0...v6.0.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these
updates again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/rustymotors/server).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6ImRldiIsImxhYmVscyI6W119-->
This pull request was closed.
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