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.

Sign up for GitHub

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

Jump to bottom

refactor(turbopack): Use ResolvedVc<T> for struct fields in next-api, final part #73367

Merged
merged 12 commits into from
Dec 8, 2024
Merged

refactor(turbopack): Use ResolvedVc<T> for struct fields in next-api, final part #73367

merged 12 commits into from
Dec 8, 2024
+80 −66

Conversation

kdy1
Copy link
Member

@kdy1 kdy1 commented Nov 29, 2024 •
edited by padmaia
Loading

Closes PACK-3622

@ijjk ijjk added created-by: Turbopack team PRs by the Turbopack team. Turbopack Related to Turbopack with Next.js. labels Nov 29, 2024
This was referenced Nov 29, 2024
Merged
Merged
Merged
Merged
@kdy1 Graphite App
Copy link
Member Author

kdy1 commented Nov 29, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@kdy1 kdy1 mentioned this pull request Nov 29, 2024
Merged
@kdy1 kdy1 changed the title fix build refactor(turbopack): Use ResolvedVc<T> for struct fields in next-api, final part Nov 29, 2024
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-turbopack-core branch from 8f8fcbe to cde5aa6 Compare November 29, 2024 23:53
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 30c18ae to e4482ba Compare November 29, 2024 23:53
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-turbopack-core branch from cde5aa6 to 8e62f48 Compare November 29, 2024 23:59
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch 3 times, most recently from 81a09af to 2a196b0 Compare November 30, 2024 01:35
This was referenced Nov 30, 2024
Merged
Merged
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 2a196b0 to 395dfe5 Compare November 30, 2024 01:37
@kdy1 kdy1 changed the base branch from kdy1/resolved-vc-turbopack-core to kdy1/resolved-vc-trivial November 30, 2024 01:37
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 395dfe5 to 847a1b4 Compare November 30, 2024 01:39
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-trivial branch 2 times, most recently from 3c7514b to fbedd42 Compare November 30, 2024 02:22
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 847a1b4 to c2244c1 Compare November 30, 2024 02:22
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-trivial branch from fbedd42 to fd14fcf Compare November 30, 2024 06:41
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from c2244c1 to b30875f Compare November 30, 2024 06:49
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-trivial branch from f5dfdbe to 09a670c Compare November 30, 2024 09:03
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from b30875f to 3bec3b2 Compare November 30, 2024 09:04
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-trivial branch from 169fbb8 to baed0b1 Compare December 3, 2024 01:58
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 3bec3b2 to c1474b9 Compare December 3, 2024 01:58
@kdy1 kdy1 mentioned this pull request Dec 3, 2024
Merged
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-extras branch from 710fa56 to 3fb1e4c Compare December 5, 2024 06:42
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 79bf67c to 518ba00 Compare December 5, 2024 06:43
mischnic
mischnic reviewed Dec 5, 2024
View reviewed changes
crates/next-api/src/route.rs
*data_endpoint = data_endpoint.resolve().await?;
if let Route::AppPage(routes) = self {
for route in routes {
route.resolve().await?;
Copy link
Contributor

@mischnic mischnic Dec 5, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

We need to change AppPageRoute to contain ResolvedVc as well

Then this whole function can be removed

@kdy1 kdy1 force-pushed the kdy1/resolved-vc-extras branch 2 times, most recently from 2e94641 to 20b70e5 Compare December 5, 2024 08:31
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 518ba00 to 335ea4e Compare December 5, 2024 08:31
@kdy1 kdy1 force-pushed the kdy1/resolved-vc-extras branch from 20b70e5 to 113bdb7 Compare December 8, 2024 04:57
@kdy1 kdy1 force-pushed the _kdy1/resolved-vc-next-api-last branch from 335ea4e to 111fba3 Compare December 8, 2024 04:57
@kdy1 kdy1 changed the base branch from kdy1/resolved-vc-extras to graphite-base/73367 December 8, 2024 07:41
@kdy1 Graphite App
Copy link
Member Author

kdy1 commented Dec 8, 2024 •
edited
Loading

Merge activity

  • Dec 8, 4:43 PM GMT+9: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..
  • Dec 8, 5:26 PM GMT+9: A user merged this pull request with Graphite.

@kdy1 kdy1 force-pushed the graphite-base/73367 branch from 113bdb7 to 6b9baaa Compare December 8, 2024 07:46
@kdy1 kdy1 changed the base branch from graphite-base/73367 to canary December 8, 2024 07:46
@kdy1 kdy1 merged commit f348241 into canary Dec 8, 2024
110 of 112 checks passed
@kdy1 kdy1 deleted the _kdy1/resolved-vc-next-api-last branch December 8, 2024 08:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 22, 2024
@kdy1 kdy1 self-assigned this Feb 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mischnic mischnic

@bgw bgw

Assignees

@kdy1 kdy1

Labels
created-by: Turbopack team PRs by the Turbopack team. locked Turbopack Related to Turbopack with Next.js.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@kdy1 @ijjk @bgw @mischnic