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

fix: make build:review compatible with 🥔s and 🧮s #1598

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

alecdwm
Copy link
Member

@alecdwm alecdwm commented Sep 4, 2024

Previously:

  • Use docker buildx to install deps & build the .zip file for the release

Now:

  • Use docker build (legacy builder) to install deps to an image, then use docker run a container using the image (with ./review bind-mounted to the host) to build the .zip file for the release

@alecdwm alecdwm enabled auto-merge (squash) September 4, 2024 12:26
@alecdwm alecdwm force-pushed the fix/potato-abacus-review-compat branch from 3d3700c to 83ecaa7 Compare September 4, 2024 12:29
@alecdwm alecdwm force-pushed the fix/potato-abacus-review-compat branch from 279b430 to cfe1999 Compare September 17, 2024 01:21
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] None +2 108 kB feross
npm/[email protected] None 0 283 kB mikemcl
npm/[email protected] environment, filesystem 0 71.6 kB motdotla
npm/[email protected] Transitive: environment, filesystem +1 576 kB vitaly
npm/[email protected] None 0 1.41 MB bnjmnt4n
npm/[email protected] None 0 176 kB pieroxy
npm/[email protected] None 0 24.4 kB ai
npm/[email protected] environment, filesystem, unsafe 0 11.2 MB prettier-bot
npm/[email protected] environment +3 4.62 MB gnoff
npm/[email protected] environment 0 24.8 kB gaearon
npm/[email protected] environment +2 337 kB gnoff
npm/[email protected] None +1 4.59 MB blesh
npm/[email protected] None +2 124 kB npm-cli-ops
npm/[email protected] None 0 144 kB binarymuse
npm/[email protected] None 0 116 kB ctavan

🚮 Removed packages: npm/@openzeppelin/[email protected], npm/@peculiar/[email protected], npm/@polkadot-api/[email protected], npm/@polkadot-api/[email protected], npm/@polkadot-api/[email protected], npm/@polkadot-api/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@polkadot/[email protected], npm/@scure/[email protected], npm/@scure/[email protected], npm/@sentry/[email protected], npm/@sentry/[email protected], npm/@sentry/[email protected], npm/@solana/[email protected], npm/@spruceid/[email protected], npm/@substrate/[email protected], npm/@supercharge/[email protected], npm/@svgr/[email protected], npm/@swc/[email protected], npm/@swc/[email protected], npm/@swc/[email protected], npm/@tailwindcss/[email protected], npm/@tailwindcss/[email protected], npm/@talismn/[email protected], npm/@talismn/[email protected], npm/@talismn/[email protected], npm/@tanstack/[email protected], npm/@testing-library/[email protected], npm/@testing-library/[email protected], npm/@testing-library/[email protected], npm/@testing-library/[email protected], npm/@testing-library/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@vitejs/[email protected], npm/@vitejs/[email protected]

View full report↗︎

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.

1 participant