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

Add improved parent type #31

Merged
merged 3 commits into from Sep 23, 2021
Merged

Add improved parent type #31

merged 3 commits into from Sep 23, 2021

Conversation

wooorm
Copy link
Member

@wooorm wooorm commented Sep 21, 2021

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

Add improved parent type

Previously, a basic Parent from @types/unist was used for the third parameter of a visitor (parent). This changes that to instead use an array of descendants in tree which implement the abstract Parent interface and can have node as a child.

Closes GH-30.
Related-to: syntax-tree/unist-util-visit-parents#11.

/cc @crossjs

Previously, a basic `Parent` from `@types/unist` was used for the third
parameter of a visitor (`parent`).
This changes that to instead use an array of descendants in `tree`
which implement the abstract `Parent` interface and can have `node`
as a child.

Closes GH-30.
Related-to:  syntax-tree/unist-util-visit-parents#11.
@wooorm wooorm added 🦋 type/enhancement This is great to have 🧒 semver/minor This is backwards-compatible change ☂️ area/types This affects typings 👍 phase/yes Post is accepted and can be worked on labels Sep 21, 2021
@github-actions

This comment has been minimized.

@wooorm wooorm changed the title Update dev-dependencies Add improved parent type Sep 21, 2021
@wooorm wooorm merged commit a76200b into main Sep 23, 2021
@wooorm wooorm deleted the parent branch September 23, 2021 13:45
@wooorm wooorm added the 💪 phase/solved Post is done label Sep 23, 2021
@github-actions

This comment has been minimized.

@github-actions github-actions bot removed the 👍 phase/yes Post is accepted and can be worked on label Sep 23, 2021
kodiakhq bot pushed a commit to X-oss-byte/Nextjs that referenced this pull request Oct 16, 2023
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [unist-util-visit](https://togithub.com/syntax-tree/unist-util-visit) | [`2.0.3` -> `5.0.0`](https://renovatebot.com/diffs/npm/unist-util-visit/2.0.3/5.0.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/unist-util-visit/5.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/unist-util-visit/5.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/unist-util-visit/2.0.3/5.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/unist-util-visit/2.0.3/5.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>syntax-tree/unist-util-visit (unist-util-visit)</summary>

### [`v5.0.0`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/5.0.0)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/4.1.2...5.0.0)

##### Change

-   [`4dcff31`](https://togithub.com/syntax-tree/unist-util-visit/commit/4dcff31) Update `@types/unist`
    **migrate**: update too
-   [`befc0b3`](https://togithub.com/syntax-tree/unist-util-visit/commit/befc0b3) Change to require Node.js 16
    **migrate**: update too
-   [`b5f36de`](https://togithub.com/syntax-tree/unist-util-visit/commit/b5f36de) Change to use `export` map
    **migrate**: don’t use private APIs
-   [`89fc050`](https://togithub.com/syntax-tree/unist-util-visit/commit/89fc050) Change to remove `complex-types.d.ts`
    **migrate**: use main export
-   [`12c9ee9`](https://togithub.com/syntax-tree/unist-util-visit/commit/12c9ee9) Change to pass `undefined`, not `null`
    **migrate**: change `null` to `undefined`

##### Fix

-   [`3cb2732`](https://togithub.com/syntax-tree/unist-util-visit/commit/3cb2732) Fix performance of `InclusiveDescendant` type

**Full Changelog**: syntax-tree/unist-util-visit@4.1.2...5.0.0

### [`v4.1.2`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/4.1.2)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/4.1.1...4.1.2)

##### Misc

-   [`332b6e0`](https://togithub.com/syntax-tree/unist-util-visit/commit/332b6e0) Add improved docs
-   [`754d038`](https://togithub.com/syntax-tree/unist-util-visit/commit/754d038) [`9f4813b`](https://togithub.com/syntax-tree/unist-util-visit/commit/9f4813b) Refactor code-style
-   [`69cbec6`](https://togithub.com/syntax-tree/unist-util-visit/commit/69cbec6) Update `tsconfig.json`

**Full Changelog**: syntax-tree/unist-util-visit@4.1.1...4.1.2

### [`v4.1.1`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/4.1.1)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/4.1.0...4.1.1)

-   [`6742b2d`](https://togithub.com/syntax-tree/unist-util-visit/commit/6742b2d) Fix TypeScript `node16` and ESM
    by [@&#8203;alecmev](https://togithub.com/alecmev) in [syntax-tree/unist-util-visit#35
-   [`6789ad5`](https://togithub.com/syntax-tree/unist-util-visit/commit/6789ad5) Add improved docs
-   [`d190ee7`](https://togithub.com/syntax-tree/unist-util-visit/commit/d190ee7) Fix typo
    by [@&#8203;jangxyz](https://togithub.com/jangxyz) in [syntax-tree/unist-util-visit#34

**Full Changelog**: syntax-tree/unist-util-visit@4.1.0...4.1.1

### [`v4.1.0`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/4.1.0)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/4.0.0...4.1.0)

-   [`a76200b`](https://togithub.com/syntax-tree/unist-util-visit/commit/a76200b) Add improved parent type
    by [@&#8203;wooorm](https://togithub.com/wooorm) in [syntax-tree/unist-util-visit#31

**Full Changelog**: syntax-tree/unist-util-visit@4.0.0...4.1.0

### [`v4.0.0`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/4.0.0)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/3.1.0...4.0.0)

-   [`050fdc3`](https://togithub.com/syntax-tree/unist-util-visit/commit/050fdc3) Change types to base what `visitor` gets on `tree`
    (**breaking** if you use TypeScript, read commit for info)

### [`v3.1.0`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/3.1.0)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/3.0.1...3.1.0)

-   [`5e98354`](https://togithub.com/syntax-tree/unist-util-visit/commit/5e98354) Add `VisitorResult` type

### [`v3.0.1`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/3.0.1)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/3.0.0...3.0.1)

-   [`a14d311`](https://togithub.com/syntax-tree/unist-util-visit/commit/a14d311) Fix example to use ESM

### [`v3.0.0`](https://togithub.com/syntax-tree/unist-util-visit/releases/tag/3.0.0)

[Compare Source](https://togithub.com/syntax-tree/unist-util-visit/compare/2.0.3...3.0.0)

-   [`ed0bccd`](https://togithub.com/syntax-tree/unist-util-visit/commit/ed0bccd) Add JSDoc based types
-   [`dfbcafb`](https://togithub.com/syntax-tree/unist-util-visit/commit/dfbcafb) Use ESM
    note: change from ~~`const visit = require('unist-util-visit')`~~ to `import {visit} from 'unist-util-visit'`

</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.

---

 - [ ] 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/X-oss-byte/Nextjs).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☂️ area/types This affects typings 💪 phase/solved Post is done 🧒 semver/minor This is backwards-compatible change 🦋 type/enhancement This is great to have
Development

Successfully merging this pull request may close these issues.

None yet

1 participant