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(gatby-script): Shim requestIdleCallback #35611

Merged
merged 6 commits into from
May 17, 2022

Conversation

marvinjude
Copy link
Contributor

Description

Shim requestIdleCallback, cancelIdleCallback

[sc-48843]

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 9, 2022
@marvinjude marvinjude force-pushed the shim-request-idle-callback branch 2 times, most recently from 10538ca to 84ff088 Compare May 11, 2022 07:20
@LekoArts LekoArts added topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters) and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels May 11, 2022
@marvinjude marvinjude marked this pull request as ready for review May 15, 2022 18:15
@marvinjude marvinjude force-pushed the shim-request-idle-callback branch from ab5cef2 to e55e787 Compare May 16, 2022 08:38
@marvinjude marvinjude added this to the Script Component milestone May 16, 2022
Copy link
Contributor

@tyhopp tyhopp left a comment

Choose a reason for hiding this comment

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

Few changes, please make sure the tests pass locally and in CI.

You can update the outdated load plugin snapshots with yarn jest /Users/<YOUR_USER>/Developer/gatsby/gatsby/packages/gatsby/src/bootstrap/load-plugins/__tests__ --updateSnapshot so that unit tests pass and you can see these e2e tests pass in CI.

@marvinjude marvinjude force-pushed the shim-request-idle-callback branch from 3582825 to ba5591c Compare May 16, 2022 11:48
@marvinjude
Copy link
Contributor Author

Few changes, please make sure the tests pass locally and in CI.

You can update the outdated load plugin snapshots with yarn jest /Users/<YOUR_USER>/Developer/gatsby/gatsby/packages/gatsby/src/bootstrap/load-plugins/__tests__ --updateSnapshot so that unit tests pass and you can see these e2e tests pass in CI.

Snapshot updates are now on feat-script-component

@tyhopp tyhopp mentioned this pull request May 17, 2022
12 tasks
@tyhopp tyhopp force-pushed the shim-request-idle-callback branch from 83fb9f3 to f5d616a Compare May 17, 2022 03:57
@tyhopp tyhopp added type: feature or enhancement Issue that is not a bug and requests the addition of a new feature or enhancement. and removed topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters) labels May 17, 2022
@tyhopp tyhopp merged commit cfd280f into feat-script-component May 17, 2022
@tyhopp tyhopp deleted the shim-request-idle-callback branch May 17, 2022 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature or enhancement Issue that is not a bug and requests the addition of a new feature or enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants