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

feat: Check the number of direct dependants while tree shaking #8473

Merged
merged 3 commits into from
Jun 14, 2024

Conversation

kdy1
Copy link
Member

@kdy1 kdy1 commented Jun 13, 2024

Description

image

In the graph above, x and y can both reach item 5, but grouping from item 5 to item 9 is fine

Testing Instructions

@kdy1 kdy1 self-assigned this Jun 13, 2024
@kdy1 kdy1 requested a review from a team as a code owner June 13, 2024 05:36
Copy link

vercel bot commented Jun 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 13, 2024 5:37am
rust-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 13, 2024 5:37am
8 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-designsystem-docs ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-gatsby-web ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-kitchensink-blog ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-native-web ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-svelte-web ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-tailwind-web ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am
examples-vite-web ⬜️ Ignored (Inspect) Jun 13, 2024 5:37am

Copy link
Contributor

🟢 Turbopack Benchmark CI successful 🟢

Thanks

Copy link
Contributor

✅ This change can build next-swc

Copy link
Contributor

github-actions bot commented Jun 13, 2024

⚠️ CI failed ⚠️

The following steps have failed in CI:

  • Turbopack Rust tests (mac/win, non-blocking)

See workflow summary for details

N8["Items: [ItemId(4, Normal)]"];
N9["Items: [ItemId(5, VarDeclarator(0))]"];
N10["Items: [ItemId(6, Normal)]"];
N11["Items: [ItemId(0, ImportBinding(0)), ItemId(9, Normal)]"];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@kdy1 kdy1 merged commit 5336fd6 into main Jun 14, 2024
56 of 58 checks passed
@kdy1 kdy1 deleted the kdy1/tree-shaking-group branch June 14, 2024 12:51
kdy1 added a commit to vercel/next.js that referenced this pull request Jun 18, 2024
# Turbopack

* vercel/turborepo#8473 <!-- Donny/강동윤 - feat: Check
the number of direct dependants while tree shaking -->
* vercel/turborepo#8420 <!-- Tobias Koppers - Tree
Shaking grouping of items -->
* vercel/turborepo#8510 <!-- Tobias Koppers - read
larger chunks when reading a trace file -->
* vercel/turborepo#8509 <!-- Tobias Koppers - avoid
duplicates in primary_modules -->
* vercel/turborepo#8508 <!-- Tobias Koppers - only
create spans for nodes that are actually visited -->
* vercel/turborepo#8502 <!-- Donny/강동윤 - build:
Update `swc_core` to `v0.95.2` -->


### What?

Update `swc_core`.

- [CHANGELOG](https://github.com/swc-project/swc/blob/064af5391d8da9599f277bd68fd0b479b0b1235c/CHANGELOG.md#160---2024-06-15)


### Why?

I improved the performance of the parser by margin.
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 25, 2024
…l/turborepo#8473)

### Description

<img width="285" alt="image" src="https://github.com/vercel/turbo/assets/29931815/c37d81fb-ab6e-482a-8e6f-3784a62e923b">

In the graph above, `x` and `y` can both reach `item 5`, but grouping from `item 5` to `item 9` is fine
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 29, 2024
…l/turborepo#8473)

### Description

<img width="285" alt="image" src="https://github.com/vercel/turbo/assets/29931815/c37d81fb-ab6e-482a-8e6f-3784a62e923b">

In the graph above, `x` and `y` can both reach `item 5`, but grouping from `item 5` to `item 9` is fine
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 29, 2024
…l/turborepo#8473)

### Description

<img width="285" alt="image" src="https://github.com/vercel/turbo/assets/29931815/c37d81fb-ab6e-482a-8e6f-3784a62e923b">

In the graph above, `x` and `y` can both reach `item 5`, but grouping from `item 5` to `item 9` is fine
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 1, 2024
…l/turborepo#8473)

### Description

<img width="285" alt="image" src="https://github.com/vercel/turbo/assets/29931815/c37d81fb-ab6e-482a-8e6f-3784a62e923b">

In the graph above, `x` and `y` can both reach `item 5`, but grouping from `item 5` to `item 9` is fine
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 14, 2024
* vercel/turborepo#8473 <!-- Donny/강동윤 - feat: Check
the number of direct dependants while tree shaking -->
* vercel/turborepo#8420 <!-- Tobias Koppers - Tree
Shaking grouping of items -->
* vercel/turborepo#8510 <!-- Tobias Koppers - read
larger chunks when reading a trace file -->
* vercel/turborepo#8509 <!-- Tobias Koppers - avoid
duplicates in primary_modules -->
* vercel/turborepo#8508 <!-- Tobias Koppers - only
create spans for nodes that are actually visited -->
* vercel/turborepo#8502 <!-- Donny/강동윤 - build:
Update `swc_core` to `v0.95.2` -->

Update `swc_core`.

- [CHANGELOG](https://github.com/swc-project/swc/blob/064af5391d8da9599f277bd68fd0b479b0b1235c/CHANGELOG.md#160---2024-06-15)

I improved the performance of the parser by margin.
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 15, 2024
* vercel/turborepo#8473 <!-- Donny/강동윤 - feat: Check
the number of direct dependants while tree shaking -->
* vercel/turborepo#8420 <!-- Tobias Koppers - Tree
Shaking grouping of items -->
* vercel/turborepo#8510 <!-- Tobias Koppers - read
larger chunks when reading a trace file -->
* vercel/turborepo#8509 <!-- Tobias Koppers - avoid
duplicates in primary_modules -->
* vercel/turborepo#8508 <!-- Tobias Koppers - only
create spans for nodes that are actually visited -->
* vercel/turborepo#8502 <!-- Donny/강동윤 - build:
Update `swc_core` to `v0.95.2` -->

Update `swc_core`.

- [CHANGELOG](https://github.com/swc-project/swc/blob/064af5391d8da9599f277bd68fd0b479b0b1235c/CHANGELOG.md#160---2024-06-15)

I improved the performance of the parser by margin.
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 16, 2024
# Turbopack

* vercel/turborepo#8473 <!-- Donny/강동윤 - feat: Check
the number of direct dependants while tree shaking -->
* vercel/turborepo#8420 <!-- Tobias Koppers - Tree
Shaking grouping of items -->
* vercel/turborepo#8510 <!-- Tobias Koppers - read
larger chunks when reading a trace file -->
* vercel/turborepo#8509 <!-- Tobias Koppers - avoid
duplicates in primary_modules -->
* vercel/turborepo#8508 <!-- Tobias Koppers - only
create spans for nodes that are actually visited -->
* vercel/turborepo#8502 <!-- Donny/강동윤 - build:
Update `swc_core` to `v0.95.2` -->


### What?

Update `swc_core`.

- [CHANGELOG](https://github.com/swc-project/swc/blob/064af5391d8da9599f277bd68fd0b479b0b1235c/CHANGELOG.md#160---2024-06-15)


### Why?

I improved the performance of the parser by margin.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants