Skip to content

Conversation

wraithgar
Copy link
Member

No description provided.

@wraithgar wraithgar requested a review from a team as a code owner October 8, 2025 18:11
@wraithgar
Copy link
Member Author

Something is set env.CI explicitly to false

diff --git a/node_modules/ci-info/index.js b/node_modules/ci-info/index.js
index 38056d9aa..703f754b6 100644
--- a/node_modules/ci-info/index.js
+++ b/node_modules/ci-info/index.js
@@ -15,6 +15,7 @@ exports.name = null
 exports.isPR = null
 exports.id = null
 
+console.log(`ci-info being required, env.ci ${env.CI}`)
 if (env.CI !== 'false') {
   vendors.forEach(function (vendor) {
     const envs = Array.isArray(vendor.env) ? vendor.env : [vendor.env]
$ npx tap -Rclassic --no-coverage test/publish.js -g "automatic provenance with incorrect permissions"
test/publish.js ............    ci-info being required, env.ci false
test/publish.js ...................................... 0/24

@wraithgar wraithgar force-pushed the gar/ci-info-breaking-change branch from cd8d42e to b8050e1 Compare October 8, 2025 19:06
@wraithgar wraithgar changed the title ci-info breaking change test deps: [email protected] Oct 8, 2025
@wraithgar wraithgar merged commit fa7cc6f into latest Oct 8, 2025
34 checks passed
@wraithgar wraithgar deleted the gar/ci-info-breaking-change branch October 8, 2025 19:15
@github-actions github-actions bot mentioned this pull request Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants