Repository navigation
Add the CITGM environment variable for Babel - #1213
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
|
@MikeMcC399 Babel already respects this env variable (babel/babel#18379). Is there anything to do other than what's in this PR to fix the issue? Do we need to cut a Babel release with that change, or is it enough for it to be on |
|
does not have any |
|
Ok I'm doing a Babel release now |
|
Done |
|
That looks good! I tested on a non-Jenkins system: Ubuntu 26.04.1 LTS with Node.js 24.22.1 LTS: git clone https://github.com/nodejs/citgm
cd citgm
git remote add liuxingbaoyu https://github.com/liuxingbaoyu/citgm
git fetch liuxingbaoyu env-CITGM
git checkout liuxingbaoyu/env-CITGM --track
npm install
npm link
citgm @babel/core$ citgm @babel/core
info: starting | @babel/core
info: lookup | @babel/core
info: lookup-found | @babel/core
info: @babel/core lookup-replace| https://github.com/babel/babel/archive/v8.0.9.tar.gz
info: @babel/core yarn: | Downloading project: https://github.com/babel/babel/archive/v8.0.9.tar.gz
info: @babel/core npm: | Project downloaded v8.0.9.tar.gz
info: @babel/core yarn: | yarn install started
info: @babel/core yarn: | yarn install successfully completed
info: @babel/core yarn: | test suite started
info: passing module(s) |
info: module name: | @babel/core
info: version: | 8.0.9
info: done | The smoke test has passed.
info: duration | test duration: 116953ms |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1213 +/- ##
=======================================
Coverage 96.11% 96.11%
=======================================
Files 29 29
Lines 2213 2213
=======================================
Hits 2127 2127
Misses 86 86 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Unfortunately this was merged with the wrong commit message. It should have had That could be manually corrected when release-please puts together the release notes. |
Relates to #1194
Checklist
here