jenkins-library icon indicating copy to clipboard operation
jenkins-library copied to clipboard

feat(npmExecuteScripts) add support for pnpm

Open MichaelSp opened this issue 3 years ago • 20 comments

Changes

  • Introduce a new tool field that can be used to fix the tool to one of auto,npm, yarn, pnpm.

  • Auto detection of tool based on lock file and installs tool/package manager globally if not present (npm install -g yarn/pnpm)

  • Tool specific commands for install and run For eg., npm get/set registry --> yarn get/set registry or pnpm get/set registry npm run script-name --> yarn run script-name or pnpm run script-name npm publish --> yarn/pnpm publish

  • [x] Tests

  • [x] Documentation

MichaelSp avatar May 30 '22 08:05 MichaelSp

/it

MichaelSp avatar May 30 '22 19:05 MichaelSp

/it

MichaelSp avatar May 30 '22 21:05 MichaelSp

@anilkeshav27 if you have time, can you please check and comment?

MichaelSp avatar Aug 31 '22 15:08 MichaelSp

/it

MichaelSp avatar Nov 07 '22 15:11 MichaelSp

/it

MichaelSp avatar Nov 11 '22 15:11 MichaelSp

quick question: is bom creation and publishing addressed?

If we boms cannot be created we should be very explicit in documentation that the step can only be used for certain cases. This should also be considered when thinking about the default setting, I guess.

OliverNocon avatar Nov 11 '22 15:11 OliverNocon

pnpm is currently not supported: https://www.npmjs.com/package/@cyclonedx/bom

image

MichaelSp avatar Nov 11 '22 16:11 MichaelSp