Skip to content

fix(cli): run git commit in apps directory for all commands#154

Merged
BradMclain merged 3 commits intodevelopfrom
commit-dir-consistency
May 6, 2026
Merged

fix(cli): run git commit in apps directory for all commands#154
BradMclain merged 3 commits intodevelopfrom
commit-dir-consistency

Conversation

@BradMclain
Copy link
Copy Markdown
Collaborator

tag, untag, setenv, unsetenv, and setcluster were running git commit in whatever the current working directory was meaning if gitops is run from outside the cluster repo (e.g. during local development) it commits to the wrong repo. bump and redeploy already handled this correctly. Aligns all commands to the same pattern.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
uptickmetachu
uptickmetachu previously approved these changes May 6, 2026
@uptickmetachu
Copy link
Copy Markdown
Collaborator

CI

@BradMclain
Copy link
Copy Markdown
Collaborator Author

BradMclain commented May 6, 2026

CI

mise install failing, github issue?

EDIT: nvm its python will fix it up

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Docker Images

Commit: 4d66cca59011cbb33bf03c8ef2c481c1d7c7317e

Tag
610829907584.dkr.ecr.ap-southeast-2.amazonaws.com/gitops:test-4d66cca

@BradMclain BradMclain force-pushed the commit-dir-consistency branch from b3a2d3b to c2626de Compare May 6, 2026 06:08
@BradMclain BradMclain merged commit 6f02e00 into develop May 6, 2026
3 checks passed
@BradMclain BradMclain deleted the commit-dir-consistency branch May 6, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants