Skip to content

fix local-prefix from goimports#50394

Open
mmorel-35 wants to merge 1 commit intomoby:masterfrom
mmorel-35:goimports/local-prefix
Open

fix local-prefix from goimports#50394
mmorel-35 wants to merge 1 commit intomoby:masterfrom
mmorel-35:goimports/local-prefix

Conversation

@mmorel-35
Copy link
Contributor

- What I did

Define local-prefix for goimports as github.com/docker/docker

- How I did it

Modify golangci-lint config and execute golangci-lint fmt ./...

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@mmorel-35 mmorel-35 force-pushed the goimports/local-prefix branch 5 times, most recently from 9706a50 to ba05466 Compare July 15, 2025 18:14
@corhere
Copy link
Contributor

corhere commented Jul 18, 2025

I like it! I wonder if it's worthwhile to just rip off the bandaid, so to speak, by combining this change with another gofumpt run @thaJeztah

@mmorel-35
Copy link
Contributor Author

I's like that too . There is already a lot of modification here so it's probably better to apply gofumpt in a dedicated PR.

@mmorel-35 mmorel-35 force-pushed the goimports/local-prefix branch 4 times, most recently from 05cd5b4 to 95ac239 Compare July 23, 2025 06:51
@mmorel-35 mmorel-35 force-pushed the goimports/local-prefix branch 3 times, most recently from f527dd6 to 9354880 Compare July 29, 2025 17:41
@mmorel-35 mmorel-35 force-pushed the goimports/local-prefix branch 6 times, most recently from 815e7f7 to 92c2dba Compare August 11, 2025 18:11
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
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