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

Allow deepEqual fonction to be configured globally (4.x.x branch) #1553

Merged
merged 1 commit into from Jan 5, 2024

Conversation

forty
Copy link
Contributor

@forty forty commented Dec 18, 2023

Identical change as #1551 on 4.x.x branch

@forty forty requested a review from a team as a code owner December 18, 2023 16:14
@forty forty changed the title Allow deepEqual fonction to be configured globally Allow deepEqual fonction to be configured globally (4.x.x banch) Dec 18, 2023
@forty forty changed the title Allow deepEqual fonction to be configured globally (4.x.x banch) Allow deepEqual fonction to be configured globally (4.x.x branch) Dec 18, 2023
@forty
Copy link
Contributor Author

forty commented Jan 4, 2024

Thanks @43081j for the review!

Happy new year to the Chai team and congrats on the v5 release 👏

I assume you must be busy handling "after sales" of the release, but do you think this MR could be merged and released any time soon?

Thanks!

@koddsson koddsson merged commit fb5fd85 into chaijs:4.x.x Jan 5, 2024
@koddsson
Copy link
Member

koddsson commented Jan 5, 2024

Let me see if I can publish this without breaking everything 😅

@koddsson
Copy link
Member

koddsson commented Jan 5, 2024

Should be out there now in v4.4.0

@forty
Copy link
Contributor Author

forty commented Jan 5, 2024

Awesome, thanks a lot @koddsson ! Everything looks good on my side (and you haven't override the latest tag ;) )

@koddsson
Copy link
Member

koddsson commented Jan 5, 2024

and you haven't override the latest tag ;)

Oh but I did for a couple of minutes 😭

@link2xt
Copy link

link2xt commented Jan 5, 2024

This broke our CI: deltachat/deltachat-core-rust#5153

Fails with:


> deltachat-node@1.132.1 test:mocha
> mocha -r esm node/test/test.js --growl --reporter=spec --bail --exit

notification support prerequisites not installed... cannot enable!

/home/runner/work/deltachat-core-rust/deltachat-core-rust/node_modules/chai/lib/chai/assertion.js:69
    flag(this, 'eql', config.deepEqual ?? util.eql);
                                        ^

SyntaxError: Invalid or unexpected token
    at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)
Error: Process completed with exit code 1.

CI runs node.js 18.19.0.

@forty
Copy link
Contributor Author

forty commented Jan 6, 2024

Are you sure of your nodejs version? Nullish Coalescing operator ?? exists since nodejs 14.

$ podman run -it --rm node:18.19.0 -e 'console.log(undefined ?? "ok")'
ok

@forty
Copy link
Contributor Author

forty commented Jan 6, 2024

Ok, I found the issue in your setup standard-things/esm#909

Looks like you are using this "esm" package, which have been dead for a while, last publish was 5 years ago and the repo is archived, so I would not expect a fix 😬

@link2xt
Copy link

link2xt commented Jan 6, 2024

Ok, I found the issue in your setup standard-things/esm#909

Looks like you are using this "esm" package, which have been dead for a while, last publish was 5 years ago and the repo is archived, so I would not expect a fix 😬

Thanks, I filed an issue to track this at deltachat/deltachat-core-rust#5156
For now just pinned chai to the last version that worked.

kodiakhq bot added a commit to erezrokah/aws-testing-library that referenced this pull request Jan 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 |
|---|---|---|---|---|---|
| [chai](http://chaijs.com) ([source](https://togithub.com/chaijs/chai))
| [`4.3.10` ->
`4.4.0`](https://renovatebot.com/diffs/npm/chai/4.3.10/4.4.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/chai/4.4.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/chai/4.4.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/chai/4.3.10/4.4.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/chai/4.3.10/4.4.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>chaijs/chai (chai)</summary>

### [`v4.4.0`](https://togithub.com/chaijs/chai/releases/tag/v4.4.0)

[Compare
Source](https://togithub.com/chaijs/chai/compare/v4.3.10...v4.4.0)

#### What's Changed

- Allow deepEqual fonction to be configured globally (4.x.x branch) by
[@&#8203;forty](https://togithub.com/forty) in
[chaijs/chai#1553

**Full Changelog**:
chaijs/chai@v4.3.10...v4.4.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 4am on Monday" (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 [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/erezrokah/aws-testing-library).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMjEuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEyMS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
ogkevin-robot bot added a commit to OGKevin/obsidian-kobo-highlights-import that referenced this pull request Jan 11, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [chai](http://chaijs.com) ([source](https://togithub.com/chaijs/chai)) | devDependencies | minor | [`4.3.10` -> `4.4.0`](https://renovatebot.com/diffs/npm/chai/4.3.10/4.4.0) |

---

### Release Notes

<details>
<summary>chaijs/chai (chai)</summary>

### [`v4.4.0`](https://togithub.com/chaijs/chai/releases/tag/v4.4.0)

[Compare Source](https://togithub.com/chaijs/chai/compare/v4.3.10...v4.4.0)

#### What's Changed

-   Allow deepEqual fonction to be configured globally (4.x.x branch) by [@&#8203;forty](https://togithub.com/forty) in [chaijs/chai#1553

**Full Changelog**: chaijs/chai@v4.3.10...v4.4.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 [Renovate Bot](https://togithub.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMy4zIiwidXBkYXRlZEluVmVyIjoiMzcuMTMuMyIsInRhcmdldEJyYW5jaCI6Im1hc3RlciJ9-->
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.

None yet

4 participants