Skip to content

Rename `docker_helpers` to `docker`

Steve Xuereb requested to merge rename-pkg into master

What does this MR do?

Rename docker_helpers to docker

Why was this MR needed?

Follow best practices documented in https://golang.org/doc/effective_go.html#package-names where it specifies that package names should have undercore. This also prevents comments like !1935 (merged) (comment 306011686) since it just reduces code review speed. This also prevents goimports from adding a import alias when it's not really needed.

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

!1935 (comment 306011686)

Merge request reports