Skip to content

Conversation

@pieh
Copy link
Contributor

@pieh pieh commented Aug 30, 2021

Description

FAST_DEV flag right now results in multiple copies of same logs (one for each worker), due to transitional import in

import {
IGroupedQueryIds,
runPageQueries,
runStaticQueries,
} from "../../../services"
and fact that we handle env var at import time.

This is absolutely not ideal, but we also can't easily move the code from import time path as it might be handled too late otherwise.

Related Issues

[ch35984]
#32389 (comment)

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 30, 2021
@pieh pieh removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 30, 2021
@LekoArts LekoArts added the topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine) label Aug 30, 2021
@pieh pieh merged commit e547dc3 into master Aug 30, 2021
@pieh pieh deleted the fix/pqr/fast-dev-logs-per-worker branch August 30, 2021 13:03
vladar pushed a commit that referenced this pull request Aug 31, 2021
vladar added a commit that referenced this pull request Aug 31, 2021
)

* fix(gatsby): don't log FAST_DEV message for each worker (#32961)

(cherry picked from commit e547dc3)

* trigger ci

Co-authored-by: Michal Piechowiak <[email protected]>
Co-authored-by: Vladimir Razuvaev <[email protected]>
This was referenced Aug 31, 2021
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.

3 participants