cloudflare/wrangler-action
1
0
Fork 0
mirror of https://github.com/cloudflare/wrangler-action.git synced 2024-11-25 03:14:46 +01:00
Code Issues Projects Releases Packages Wiki Activity

Don't set packageManager default via action.yml

Browse source 
We need to distinguish between when the value is and isn't set in order to perform inference based on lockfile and only fallback to the default of npm if inference fails.
This commit is contained in:
Cina Saffary 2023-10-10 18:00:32 -05:00
parent 0e6f0d3080
commit a4509d507c
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.changeset/dirty-poets-swim.md Normal file
View file

@ -0,0 +1,5 @@
---
"wrangler-action": patch
---
Fixed the package manager not being inferred based on lockfile when the `packageManager` input isn't set.

3
action.yml
View file

@ -42,6 +42,5 @@ inputs:
description: "A string of environment variable names, separated by newlines. These will be bound to your Worker using the values of matching environment variables declared in `env` of this workflow." description: "A string of environment variable names, separated by newlines. These will be bound to your Worker using the values of matching environment variables declared in `env` of this workflow."
required: false required: false
packageManager: packageManager:
description: "The package manager you'd like to use to install and run wrangler. If not specified, a value will be inferred based on the presence of a lockfile. Valid values: [npm, pnpm, yarn, bun]" description: "The package manager you'd like to use to install and run wrangler. If not specified, the preferred package manager will be inferred based on the presence of a lockfile or fallback to using npm if no lockfile is found. Valid values are `npm` | `pnpm` | `yarn` | `bun`."
required: false required: false
default: npm