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

Ensure Corepack Usage for npm, pnpm, and yarn Command Execution #10944

Merged
Prev Previous commit
Next Next commit
fix spec
  • Loading branch information
kbukum1 committed Nov 18, 2024
commit e502b9303c3c86020c9191736630a07208078848
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,10 @@
end
let(:registry_listing_url) { "#{registry_base}/#{escaped_dependency_name}" }
let(:registry_base) { "https://registry.npmjs.org" }

# Variable to control the npm fallback version feature flag
let(:npm_fallback_version_above_v6_enabled) { false }

# Variable to control the enabling feature flag for the corepack fix
let(:enable_corepack_for_npm_and_yarn) { true }

Expand All @@ -65,6 +69,8 @@
.to_return(status: 200)
allow(Dependabot::Experiments).to receive(:enabled?)
.with(:enable_corepack_for_npm_and_yarn).and_return(enable_corepack_for_npm_and_yarn)
allow(Dependabot::Experiments).to receive(:enabled?)
.with(:npm_fallback_version_above_v6).and_return(npm_fallback_version_above_v6_enabled)
end

after do
Expand Down
Loading