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

fix(gatsby): handle cyclic chunkgroup children #38052

Merged
merged 1 commit into from
May 3, 2023

Conversation

pieh
Copy link
Contributor

@pieh pieh commented May 2, 2023

Description

Apparently this is possible (chunkgroup can be child of itself ...):
image

So this PR adds a way to break ChunkGroup children cycles when figuring out compund hash for templates (when visiting lazy imports)

Documentation

Tests

Related Issues

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 2, 2023
@pieh pieh added topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine) and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels May 2, 2023
@LekoArts LekoArts merged commit 272dacd into master May 3, 2023
@LekoArts LekoArts deleted the fix/handle-chunkgroup-cycles branch May 3, 2023 07:20
pieh added a commit that referenced this pull request May 5, 2023
pieh added a commit that referenced this pull request May 5, 2023
(cherry picked from commit 272dacd)

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants