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

Enable node 22 support #3328

Merged
merged 4 commits into from
May 5, 2024
Merged

Conversation

lenovouser
Copy link
Contributor

@lenovouser lenovouser commented Apr 30, 2024

Closes #3327

@lenovouser
Copy link
Contributor Author

I am not sure why the tests are failing on ubuntu (20/21/22), it looks like they already started doing that on 01ec280 (actions/runs/8500.../job/2328...).

Could anybody give me some guidance on how to fix that?

@novemberborn
Copy link
Member

Could anybody give me some guidance on how to fix that?

It's a Node.js bug: nodejs/node#52018

This reverts commit cd3fb46.

The issue is possibly solved in 22.1.
@novemberborn
Copy link
Member

@lenovouser
Copy link
Contributor Author

@novemberborn I did not see the UI mentioned in your linked help article, but I added you directly to the repositories 👍🏻

@novemberborn novemberborn merged commit 002b3a0 into avajs:main May 5, 2024
16 of 19 checks passed
renovate bot added a commit to smartive/zitadel-node that referenced this pull request May 5, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ava](https://avajs.dev) ([source](https://togithub.com/avajs/ava)) |
[`6.1.2` -> `6.1.3`](https://renovatebot.com/diffs/npm/ava/6.1.2/6.1.3)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/ava/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/ava/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/ava/6.1.2/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/ava/6.1.2/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>avajs/ava (ava)</summary>

### [`v6.1.3`](https://togithub.com/avajs/ava/releases/tag/v6.1.3)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.1.2...v6.1.3)

#### What's Changed

- Include Node.js 22 in supported engines and test matrix by
[@&#8203;lenovouser](https://togithub.com/lenovouser) in
[avajs/ava#3328
- Add VS Code debugging instructions for Yarn PnP projects by
[@&#8203;bitjson](https://togithub.com/bitjson) in
[avajs/ava#3317
- Document `serial` configuration option by
[@&#8203;turadg](https://togithub.com/turadg) in
[avajs/ava#3321

#### New Contributors

- [@&#8203;bitjson](https://togithub.com/bitjson) made their first
contribution in
[avajs/ava#3317
- [@&#8203;turadg](https://togithub.com/turadg) made their first
contribution in
[avajs/ava#3321
- [@&#8203;lenovouser](https://togithub.com/lenovouser) made their first
contribution in
[avajs/ava#3328

**Full Changelog**: avajs/ava@v6.1.2...v6.1.3

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 9pm,before 6am" in timezone
Europe/Zurich, 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 this update
again.

---

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

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/smartive/zitadel-node).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zNDAuMTAiLCJ1cGRhdGVkSW5WZXIiOiIzNy4zNDAuMTAiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbImRlcGVuZGVuY2llcyJdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
dubzzz pushed a commit to dubzzz/fast-check that referenced this pull request May 6, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ava](https://avajs.dev) ([source](https://togithub.com/avajs/ava)) |
[`^6.1.2` ->
`^6.1.3`](https://renovatebot.com/diffs/npm/ava/6.1.2/6.1.3) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/ava/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/ava/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/ava/6.1.2/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/ava/6.1.2/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>avajs/ava (ava)</summary>

### [`v6.1.3`](https://togithub.com/avajs/ava/releases/tag/v6.1.3)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.1.2...v6.1.3)

#### What's Changed

- Include Node.js 22 in supported engines and test matrix by
[@&#8203;lenovouser](https://togithub.com/lenovouser) in
[avajs/ava#3328
- Add VS Code debugging instructions for Yarn PnP projects by
[@&#8203;bitjson](https://togithub.com/bitjson) in
[avajs/ava#3317
- Document `serial` configuration option by
[@&#8203;turadg](https://togithub.com/turadg) in
[avajs/ava#3321

#### New Contributors

- [@&#8203;bitjson](https://togithub.com/bitjson) made their first
contribution in
[avajs/ava#3317
- [@&#8203;turadg](https://togithub.com/turadg) made their first
contribution in
[avajs/ava#3321
- [@&#8203;lenovouser](https://togithub.com/lenovouser) made their first
contribution in
[avajs/ava#3328

**Full Changelog**: avajs/ava@v6.1.2...v6.1.3

</details>

---

### Configuration

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

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

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

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

---

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

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/dubzzz/fast-check).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zNDAuMTAiLCJ1cGRhdGVkSW5WZXIiOiIzNy4zNDAuMTAiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbImRlcGVuZGVuY2llcyJdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
kodiakhq bot pushed a commit to cloudquery/plugin-sdk-javascript that referenced this pull request Jun 1, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [ava](https://avajs.dev) ([source](https://togithub.com/avajs/ava)) | devDependencies | patch | [`6.1.2` -> `6.1.3`](https://renovatebot.com/diffs/npm/ava/6.1.2/6.1.3) |

---

### Release Notes

<details>
<summary>avajs/ava (ava)</summary>

### [`v6.1.3`](https://togithub.com/avajs/ava/releases/tag/v6.1.3)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.1.2...v6.1.3)

#### What's Changed

-   Include Node.js 22 in supported engines and test matrix by [@&#8203;lenovouser](https://togithub.com/lenovouser) in [avajs/ava#3328
-   Add VS Code debugging instructions for Yarn PnP projects by [@&#8203;bitjson](https://togithub.com/bitjson) in [avajs/ava#3317
-   Document `serial` configuration option by [@&#8203;turadg](https://togithub.com/turadg) in [avajs/ava#3321

#### New Contributors

-   [@&#8203;bitjson](https://togithub.com/bitjson) made their first contribution in [avajs/ava#3317
-   [@&#8203;turadg](https://togithub.com/turadg) made their first contribution in [avajs/ava#3321
-   [@&#8203;lenovouser](https://togithub.com/lenovouser) made their first contribution in [avajs/ava#3328

**Full Changelog**: avajs/ava@v6.1.2...v6.1.3

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 4am on the first day of the month" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

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

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

---

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

---

This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zODMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjM4My4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJhdXRvbWVyZ2UiXX0=-->
aepfli pushed a commit to aepfli/markdownlint-rule-max-one-sentence-per-line that referenced this pull request Jul 8, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ava](https://avajs.dev) ([source](https://togithub.com/avajs/ava)) |
[`5.3.1` -> `6.1.3`](https://renovatebot.com/diffs/npm/ava/5.3.1/6.1.3)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/ava/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/ava/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/ava/5.3.1/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/ava/5.3.1/6.1.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>avajs/ava (ava)</summary>

### [`v6.1.3`](https://togithub.com/avajs/ava/releases/tag/v6.1.3)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.1.2...v6.1.3)

#### What's Changed

- Include Node.js 22 in supported engines and test matrix by
[@&#8203;lenovouser](https://togithub.com/lenovouser) in
[avajs/ava#3328
- Add VS Code debugging instructions for Yarn PnP projects by
[@&#8203;bitjson](https://togithub.com/bitjson) in
[avajs/ava#3317
- Document `serial` configuration option by
[@&#8203;turadg](https://togithub.com/turadg) in
[avajs/ava#3321

#### New Contributors

- [@&#8203;bitjson](https://togithub.com/bitjson) made their first
contribution in
[avajs/ava#3317
- [@&#8203;turadg](https://togithub.com/turadg) made their first
contribution in
[avajs/ava#3321
- [@&#8203;lenovouser](https://togithub.com/lenovouser) made their first
contribution in
[avajs/ava#3328

**Full Changelog**: avajs/ava@v6.1.2...v6.1.3

### [`v6.1.2`](https://togithub.com/avajs/ava/releases/tag/v6.1.2)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.1.1...v6.1.2)

#### What's Changed

- Fix throws assertions rejecting falsy values when `any: true` by
[@&#8203;gibson042](https://togithub.com/gibson042) in
[avajs/ava#3313

**Full Changelog**: avajs/ava@v6.1.1...v6.1.2

### [`v6.1.1`](https://togithub.com/avajs/ava/releases/tag/v6.1.1)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.1.0...v6.1.1)

#### What's Changed

- Fix 'previous failures' in watch mode always incrementing by
[@&#8203;novemberborn](https://togithub.com/novemberborn) in
[avajs/ava#3297

**Full Changelog**: avajs/ava@v6.1.0...v6.1.1

### [`v6.1.0`](https://togithub.com/avajs/ava/releases/tag/v6.1.0)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.0.1...v6.1.0)

##### What's Changed

- Implement `registerCompletionHandler()` by
[@&#8203;novemberborn](https://togithub.com/novemberborn) in
[avajs/ava#3283

AVA 6 expects test code to clean up gracefully when the tests are
complete, allowing the test environment (worker thread or child process)
to exit. If this does not happen, AVA will report a timeout. You can use
`registerCompletionHandler()` to perform any other clean up (or indeed
exit the process) after AVA has completed the test execution. See [the
description
here](https://togithub.com/avajs/ava/blob/main/docs/08-common-pitfalls.md#timeouts-because-a-file-failed-to-exit).
- Fix potential bug with watch mode when no failed test files are
written by [@&#8203;novemberborn](https://togithub.com/novemberborn) in
[avajs/ava#3287
- Fix `ava/internal` ESM type module by
[@&#8203;codetheweb](https://togithub.com/codetheweb) in
[avajs/ava#3292

**Full Changelog**: avajs/ava@v6.0.1...v6.1.0

### [`v6.0.1`](https://togithub.com/avajs/ava/releases/tag/v6.0.1)

[Compare Source](https://togithub.com/avajs/ava/compare/v6.0.0...v6.0.1)

#### What's Changed

- Ensure watch mode works outside of its integration tests by
[@&#8203;novemberborn](https://togithub.com/novemberborn) in
[avajs/ava#3273

**Full Changelog**: avajs/ava@v6.0.0...v6.0.1

### [`v6.0.0`](https://togithub.com/avajs/ava/releases/tag/v6.0.0)

[Compare Source](https://togithub.com/avajs/ava/compare/v5.3.1...v6.0.0)

##### Breaking Changes

- AVA now requires Node.js versions 18.18, 20.8 or 21. Versions 14 and
16 are no longer supported.
[#&#8203;3251](https://togithub.com/avajs/ava/issues/3251)
[#&#8203;3216](https://togithub.com/avajs/ava/issues/3216)

- When tests finish, worker threads or child processes are no longer
exited through `proces.exit()`. If your test file does not exit on its
own, the test run will time out.
[#&#8203;3260](https://togithub.com/avajs/ava/issues/3260)

- Changes to watch mode
[#&#8203;3218](https://togithub.com/avajs/ava/issues/3218):
- Watch mode can no longer be started via the `ava.config.*` or
`package.json` configuration.
- The `ignoredByWatcher` configuration has moved to the `watchMode`
object, under the `ignoreChanges` key.
- Watch mode now uses the built-in
[`fs.watch()`](https://nodejs.org/api/fs.html#fswatchfilename-options-listener)
in recursive mode. This is supported on Linux in Node.js 20 or newer,
and MacOS and Windows in Node.js 18 as well. There are
[caveats](https://nodejs.org/api/fs.html#caveats) to keep in mind.

- Failed assertions now throw, meaning that any subsequent code is not
executed. This also impacts the type definitions.
[#&#8203;3246](https://togithub.com/avajs/ava/issues/3246)

- [Only native
errors](https://nodejs.org/api/util.html#utiltypesisnativeerrorvalue)
are now considered errors by the `t.throws()` and `t.throwsAsync()`
assertions. [`Object.create(Error.prototype)` is **not** a native
error](Object.create\(Error.prototype\)).
[#&#8203;3229](https://togithub.com/avajs/ava/issues/3229)

- Changes to modules loaded through the `require` configuration
[#&#8203;3184](https://togithub.com/avajs/ava/issues/3184):
- If such modules export a default function, this function is now
invoked.
    -   Local files are loaded through `@ava/typescript` if necessary.

##### Improvements

##### Rewritten watcher

The watcher has been rewritten. It’s now built on
[`fs.watch()`](https://nodejs.org/api/fs.html#fswatchfilename-options-listener)
in recursive mode.

[`@vercel/nft`](https://togithub.com/vercel/nft) is used to perform
static dependency analysis, supporting ESM and CJS imports for
JavaScript & TypeScript source files. This is a huge improvement over
the previous runtime tracking of CJS imports, which did not support ESM.

Integration with
[`@ava/typescript`](https://togithub.com/avajs/typescript) has been
improved. The watcher can now detect a change to a TypeScript source
file, then wait for the corresponding build output to change before
re-running tests.

The ignoredByWatcher configuration has moved to the watchMode object,
under the ignoreChanges key.

See [#&#8203;3218](https://togithub.com/avajs/ava/issues/3218) and
[#&#8203;3257](https://togithub.com/avajs/ava/issues/3257).

##### Failed assertions now throw

Assertions now throw a `TestFailure` error when they fail. This error is
not exported or documented and should not be used or thrown manually.
You cannot catch this error in order to recover from a failure, use
`t.try()` instead.

All assertions except for `t.throws()` and `t.throwsAsync()` now return
`true` when they pass. This is useful for some of the assertions in
TypeScript where they can be used as a type guard.

Committing a failed `t.try()` result now also throws.

See [#&#8203;3246](https://togithub.com/avajs/ava/issues/3246).

##### `t.throws()` and `t.throwsAsync()` can now expect any error

By default, the thrown error (or rejection reason) must be a native
error. You can change the assertion to expect any kind of error by
setting `any: true` in the expectation object:

```js
t.throws(() => { throw 'error' }, {any: true})
```

See [#&#8203;3245](https://togithub.com/avajs/ava/issues/3245) by
[@&#8203;adiSuper94](https://togithub.com/adiSuper94).

##### The `require` configuration is now more powerful

It now loads ES modules.

Local files are loaded through `@ava/typescript` if necessary, so you
can also write these in TypeScript.

If there is a default export function, it is invoked after loading. The
function is awaited so it can do asynchronous setup before further
modules are loaded. Arguments from the configuration can be passed to
the function (as a \[[structured
clone](https://developer.mozilla.org/en-US/docs/Web/API/structuredClone)]\(https://developer.mozilla.org/en-US/docs/Web/API/structuredClone)).

See [#&#8203;3184](https://togithub.com/avajs/ava/issues/3184) by
[@&#8203;sculpt0r](https://togithub.com/sculpt0r).

##### Other changes worth noting

- Internal events can now be observed (experimentally). See
[#&#8203;3247](https://togithub.com/avajs/ava/issues/3247) by
[@&#8203;codetheweb](https://togithub.com/codetheweb). It’s experimental
and undocumented.
- You can now use `t.timeout.clear()` to restore a previous
`t.timeout()`.
[#&#8203;3221](https://togithub.com/avajs/ava/issues/3221)
- Code coverage is flushed to disk at opportune moments.
[#&#8203;3220](https://togithub.com/avajs/ava/issues/3220)

##### New Contributors

- [@&#8203;sculpt0r](https://togithub.com/sculpt0r) made their first
contribution in
[avajs/ava#3184
- [@&#8203;ZachHaber](https://togithub.com/ZachHaber) made their first
contribution in
[avajs/ava#3233
- [@&#8203;adiSuper94](https://togithub.com/adiSuper94) made their first
contribution in
[avajs/ava#3245
- [@&#8203;bricker](https://togithub.com/bricker) made their first
contribution in
[avajs/ava#3250

**Full Changelog**: avajs/ava@v5.3.1...v6.0.0

</details>

---

### Configuration

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

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

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

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

---

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

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/aepfli/markdownlint-rule-max-one-sentence-per-line).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44MS4zIiwidXBkYXRlZEluVmVyIjoiMzcuMzQwLjEwIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

Add support for Node 22
2 participants