Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork6.6k
Globals cleanup: avoid setting protection symbol when feature is off#15684
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Globals cleanup: avoid setting protection symbol when feature is off#15684
Uh oh!
There was an error while loading.Please reload this page.
Conversation
netlifybot commentedJun 18, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview forjestjs ready!Builtwithout sensitive environment variables
To edit notification comments on pull requests, go to yourNetlify project configuration. |
pkg-pr-newbot commentedJun 18, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
babel-jestbabel-plugin-jest-hoistbabel-preset-jestcreate-jest@jest/diff-sequencesexpect@jest/expect-utilsjestjest-changed-filesjest-circusjest-clijest-config@jest/console@jest/core@jest/create-cache-key-functionjest-diffjest-docblockjest-each@jest/environmentjest-environment-jsdom@jest/environment-jsdom-abstractjest-environment-node@jest/expect@jest/fake-timers@jest/get-type@jest/globalsjest-haste-mapjest-jasmine2jest-leak-detectorjest-matcher-utilsjest-message-utiljest-mock@jest/patternjest-phabricatorjest-regex-util@jest/reportersjest-resolvejest-resolve-dependenciesjest-runnerjest-runtime@jest/schemasjest-snapshot@jest/snapshot-utils@jest/source-map@jest/test-result@jest/test-sequencer@jest/transform@jest/typesjest-utiljest-validatejest-watcherjest-workerpretty-formatcommit: |
40a9867 to760523aCompare… 'globalsCleanup' configuration
760523a to429cf37Compare@cpojer not sure if you saw but this PR is ready for review :) (I can't seem to be able to use the "request review" feature) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thank you, I'll publish a new release.
33820ad intojestjs:mainUh oh!
There was an error while loading.Please reload this page.
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Uh oh!
There was an error while loading.Please reload this page.
Summary
Fixes#15678.
First commit also makes sure to run
jest-environment-nodetests will every combination of theglobalsCleanupconfiguration.