Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(gatsby): invite people with long develop bundling times to try the lazy dev js bundling feature #28116

Merged
merged 5 commits into from
Nov 17, 2020

Conversation

KyleAMathews
Copy link
Contributor

@KyleAMathews KyleAMathews commented Nov 17, 2020

It checks if the develop activity is over 20 seconds and then shows a message in the console.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Nov 17, 2020
Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

Few suggestions 👍

@pvdz pvdz marked this pull request as draft November 17, 2020 10:43
@pvdz pvdz added topic: dev tooling and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Nov 17, 2020
@KyleAMathews KyleAMathews marked this pull request as ready for review November 17, 2020 18:36
@KyleAMathews KyleAMathews changed the title feat(gatsby): WIP change to invite people with long develop bundling times to try the lazy dev js bundling feature feat(gatsby): invite people with long develop bundling times to try the lazy dev js bundling feature Nov 17, 2020
Copy link
Contributor

@sidharthachatterjee sidharthachatterjee left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thank you!

@sidharthachatterjee sidharthachatterjee added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Nov 17, 2020
@gatsbybot gatsbybot merged commit a737ea7 into master Nov 17, 2020
@delete-merged-branch delete-merged-branch bot deleted the invite-people branch November 17, 2020 19:06
@@ -26,7 +26,7 @@ export function showExperimentNoticeAfterTimeout(
}

const noticeTimeout = setTimeout(() => {
reporter.info(noticeText)
reporter.info(`\n\n${noticeText}\n\n`)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pieh a little spacing around the messages makes them stand out better. Without it they seemed lost in the logs

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants