Skip to content

[DEV-9348] Fix bug caused by missing checkouts#115

Merged
facetoe merged 1 commit into
mainfrom
DEV-9348-additional-checkouts
Jul 21, 2026
Merged

[DEV-9348] Fix bug caused by missing checkouts#115
facetoe merged 1 commit into
mainfrom
DEV-9348-additional-checkouts

Conversation

@facetoe

@facetoe facetoe commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

We moved the checkout from the setup Python step, and missed adding it to these jobs so they do not work as expected.

@facetoe
facetoe requested a review from alourie July 21, 2026 10:21
alourie
alourie previously approved these changes Jul 21, 2026

@alourie alourie 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.

Approved, but small comment - need to fix secrets reference

Comment thread .github/workflows/helm-charts-release.yml Outdated
We moved the checkout from the setup Python step, and missed adding it
to these jobs so they do not work as expected.

Also fix bug where it should be secret, not secrets when referencing
CI_GITHUB_TOKEN.

Signed-off-by: William Coe <william.coe@zepben.com>
@facetoe
facetoe force-pushed the DEV-9348-additional-checkouts branch from 3f37db4 to c9ccfea Compare July 21, 2026 23:04
@facetoe
facetoe merged commit 3faf393 into main Jul 21, 2026
1 check passed
@facetoe
facetoe deleted the DEV-9348-additional-checkouts branch July 21, 2026 23:45
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