Skip to content
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

re-enable go format checking in CI #11363

Merged
merged 3 commits into from
Sep 14, 2023
Merged

re-enable go format checking in CI #11363

merged 3 commits into from
Sep 14, 2023

Conversation

adleong
Copy link
Member

@adleong adleong commented Sep 11, 2023

A git related "dubious ownership" error was preventing the go format action from running in CI. As a result of go formatting not getting checked in CI, some go formatting drift has been introduced.

Add the appropriate git config command to resolve dubious ownership so that go format checking is run in CI.

@adleong adleong requested a review from a team as a code owner September 11, 2023 23:56
@adleong
Copy link
Member Author

adleong commented Sep 12, 2023

Signed-off-by: Alex Leong <[email protected]>
Signed-off-by: Alex Leong <[email protected]>
@adleong adleong merged commit fe9532b into main Sep 14, 2023
37 checks passed
@adleong adleong deleted the alex/fmt branch September 14, 2023 20:13
adamshawvipps pushed a commit to adamshawvipps/linkerd2 that referenced this pull request Sep 18, 2023
A git related "dubious ownership" error was preventing the go format action from running in CI. As a result of go formatting not getting checked in CI, some go formatting drift has been introduced.

Add the appropriate git config command to resolve dubious ownership so that go format checking is run in CI.

Signed-off-by: Alex Leong <[email protected]>
adamshawvipps pushed a commit to adamshawvipps/linkerd2 that referenced this pull request Sep 18, 2023
A git related "dubious ownership" error was preventing the go format action from running in CI. As a result of go formatting not getting checked in CI, some go formatting drift has been introduced.

Add the appropriate git config command to resolve dubious ownership so that go format checking is run in CI.

Signed-off-by: Alex Leong <[email protected]>
Signed-off-by: Adam Shaw <[email protected]>
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.

3 participants