Skip to content

fix(docker): ignore lifecycle scripts in actions Docker builds#481

Open
falcorocks wants to merge 4 commits into
mainfrom
falcorocks/docker-ignore-scripts
Open

fix(docker): ignore lifecycle scripts in actions Docker builds#481
falcorocks wants to merge 4 commits into
mainfrom
falcorocks/docker-ignore-scripts

Conversation

@falcorocks

@falcorocks falcorocks commented May 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add --ignore-scripts to dependency install steps in Dockerfiles
  • pnpm rebuild resolve-tspaths after install — Why: that package compiles dist/ on install; without rebuild, downstream nx build cannot run it.
  • actions-sdk build uses node …/resolve-tspaths/dist/main.jsWhy: avoids the resolve-tspaths bin shim when lifecycle scripts are disabled repo-wide.

Test plan

  • Local docker build for in-scope image target(s)
  • CI green

@falcorocks falcorocks requested review from a team as code owners May 26, 2026 14:00
@falcorocks falcorocks requested a review from lucasmoore May 26, 2026 14:00
@falcorocks falcorocks self-assigned this May 26, 2026
@netlify

netlify Bot commented May 26, 2026

Copy link
Copy Markdown

Deploy Preview for actions-ui ready!

Name Link
🔨 Latest commit 2b0d6b8
🔍 Latest deploy log https://app.netlify.com/projects/actions-ui/deploys/6a1ef1ef6eda2400087dfc2c
😎 Deploy Preview https://deploy-preview-481--actions-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jefr90 jefr90 requested a review from its-everdred May 27, 2026 16:23
@falcorocks falcorocks force-pushed the falcorocks/docker-ignore-scripts branch from b80365f to 2b0d6b8 Compare June 2, 2026 15:08
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.

2 participants