Skip to content

docker base fix: Fix Entrypoint Corruption #497

Open
eminmeydanoglu wants to merge 6 commits intomainfrom
emin/docker-base-fix
Open

docker base fix: Fix Entrypoint Corruption #497
eminmeydanoglu wants to merge 6 commits intomainfrom
emin/docker-base-fix

Conversation

@eminmeydanoglu
Copy link
Collaborator

The previous sed command incorrectly concatenated exec "$@" with source ... due to a newline escaping issue, creating the invalid command zshsource.

Fix: Switched to using sed insert mode (/exec/i ...) to safely insert the source command on a separate line.

Copy link
Member

@senceryazici senceryazici left a comment

Choose a reason for hiding this comment

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

This PR needs a rebase on #515 though

username: ${{ github.repository_owner }}
password: ${{ secrets.GITHUB_TOKEN }}

- name: Create PR manifest and push
Copy link
Member

Choose a reason for hiding this comment

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

Why not just put this step on the create manifest job above as a step? You can still use the if check since this is run: which gives you full bash access.

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