Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Dec 1, 2025

Fixes the fatal: detected dubious ownership in repository at errors we got when running git operations in post release scripts due to missing /home/byk in env. It also passes the Git identity vars for this step as they are overridden in publish repo.

Sample fail: https://github.com/getsentry/publish/actions/runs/19486535961/job/55769925123

Fixes the `fatal: detected dubious ownership in repository at` errors we got when running `git` operations in post release scripts due to missing `/home/byk` in env. It also passes the Git identity vars for this step as they are overridden in publish repo
@BYK BYK enabled auto-merge (squash) December 1, 2025 19:34
@BYK BYK requested a review from hubertdeng123 December 1, 2025 19:35
@hubertdeng123
Copy link
Member

Seems like the link you have in the PR is actually a success, this one is a failure: https://github.com/getsentry/publish/actions/runs/19833269111/job/56824349845

@BYK BYK changed the title fix: /home/byk and some others should be passed in env fix: $HOME and some others should be passed in env Dec 1, 2025
@BYK BYK disabled auto-merge December 1, 2025 19:38
@BYK BYK enabled auto-merge (squash) December 1, 2025 19:38
@BYK BYK merged commit ca9827c into master Dec 1, 2025
14 checks passed
@BYK BYK deleted the byk/fix/post-release branch December 1, 2025 19:39
@aldy505
Copy link

aldy505 commented Dec 2, 2025

Didn't expect it was a craft issue lol

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.

4 participants