Skip to content

fix(ci): [FUN-804] docker login command#561

Merged
mike-zorn merged 1 commit intomainfrom
FUN-804/mrz/release-new-ldcli
Jun 9, 2025
Merged

fix(ci): [FUN-804] docker login command#561
mike-zorn merged 1 commit intomainfrom
FUN-804/mrz/release-new-ldcli

Conversation

@mike-zorn
Copy link
Copy Markdown

@mike-zorn mike-zorn commented Jun 9, 2025

This corrects the docker login command to not accept the password via stdin. The command needs to be done on the container, not on the host.


Related Jira issue: FUN-804: Release new ldcli

Should not have accepted the AI code review suggestion 🤦‍♂️
@launchdarkly-upra launchdarkly-upra Bot changed the title fix(ci): docker login command fix(ci): [FUN-804] docker login command Jun 9, 2025
@mike-zorn mike-zorn requested review from a team and BehnH June 9, 2025 19:01
Copy link
Copy Markdown
Member

@BehnH BehnH left a comment

Choose a reason for hiding this comment

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

really hoping this is it 🤞🏻

@mike-zorn mike-zorn merged commit 5698576 into main Jun 9, 2025
8 checks passed
@mike-zorn mike-zorn deleted the FUN-804/mrz/release-new-ldcli branch June 9, 2025 19:15
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