compressed-size-action icon indicating copy to clipboard operation
compressed-size-action copied to clipboard

Allow skipping install.

Open seivan opened this issue 2 years ago • 2 comments

Using zero-install PnP with yarn doesn't play well when intalling using --frozen-lockfile. Allow opting out of installing all together.

Don't mind doing the PR if it's ok.

seivan avatar Jul 20 '22 15:07 seivan

This feature would also help when npm deps require npm ci --force for the action to work. aka deps that still don't officially support react 18.x even though things may work fine.

NateRadebaugh avatar Aug 03 '22 02:08 NateRadebaugh

Or when postinstall runs some npm scripts to help devs out, but requires devDeps and the bundles need NODE_ENV=production.

#85 & #88 add customization to the installation, but it may be better to pull this step out (do the one thing well principle)? 🤔 i.e. https://github.com/preactjs/compressed-size-action#usage becomes:

name: Compressed Size

on: [pull_request]

jobs:
  build:
    runs-on: ubuntu-latest

    steps:
      - uses: actions/checkout@v2
      - run: npm ci
      - uses: preactjs/compressed-size-action@v2

lots of variations, e.g.:

      - run: npm ci --include=dev
      # or
      - run: npm ci --ignore-scripts
      # or, for those in a tight spot
      - run: npm ci --force
      # or combinations thereof

This would also remove the need of supporting various package managers, though that support has already been added for the most used 😅.

PixnBits avatar Sep 05 '23 18:09 PixnBits

#85 probably addresses this reasonably, feel free to comment if there's something better we could land though

rschristian avatar Apr 22 '24 23:04 rschristian