Skip to content

Set safe dir for git operations in .drone.yml CI (#19641) #19643

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

techknowlogick
Copy link
Member

Our drone by necessity runs on git repositories not owned by the drone process. Unfortunately this means that git operations and thence CI builds will fail without the safe.directory option being set.

See: https://drone.gitea.io/go-gitea/gitea/54632/2/8

backport of #19641

Our drone by necessity runs on git repositories not owned by the drone process. Unfortunately this means that git operations and thence CI builds will fail without the `safe.directory` option being set. 

See: https://drone.gitea.io/go-gitea/gitea/54632/2/8
@techknowlogick techknowlogick added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label May 6, 2022
@techknowlogick techknowlogick added this to the 1.16.8 milestone May 6, 2022
@techknowlogick
Copy link
Member Author

Using 🧙 powers to merge this as it was successful in main

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 6, 2022
@techknowlogick techknowlogick merged commit 59d132f into go-gitea:release/v1.16 May 6, 2022
@techknowlogick techknowlogick deleted the backport-19641 branch May 6, 2022 18:10
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants