Skip to content

fix(docker): ignore lifecycle scripts in ecosystem Docker builds#1007

Merged
falcorocks merged 4 commits into
mainfrom
falcorocks/docker-ignore-scripts
Jun 2, 2026
Merged

fix(docker): ignore lifecycle scripts in ecosystem Docker builds#1007
falcorocks merged 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

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 13:59
@falcorocks falcorocks requested a review from tremarkley May 26, 2026 13:59
@falcorocks falcorocks self-assigned this May 26, 2026
@netlify

netlify Bot commented May 26, 2026

Copy link
Copy Markdown

Deploy Preview for superchain-playground ready!

Name Link
🔨 Latest commit f738f83
🔍 Latest deploy log https://app.netlify.com/projects/superchain-playground/deploys/6a15bf239fd86e0008ca933d
😎 Deploy Preview https://deploy-preview-1007--superchain-playground.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 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@falcorocks falcorocks merged commit 62d792f into main Jun 2, 2026
25 checks passed
@falcorocks falcorocks deleted the falcorocks/docker-ignore-scripts branch June 2, 2026 14:51
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