Skip to content

chore: update header-map module references to use docplanner namespace CORE-384#1

Open
rodrigoechaide wants to merge 6 commits intomainfrom
rename-docplanner-dcb6a
Open

chore: update header-map module references to use docplanner namespace CORE-384#1
rodrigoechaide wants to merge 6 commits intomainfrom
rename-docplanner-dcb6a

Conversation

@rodrigoechaide
Copy link
Collaborator

@rodrigoechaide rodrigoechaide commented Mar 2, 2026

Summary

  • Rename module namespace: Update the Go module path and all Traefik plugin references from rodrigoechaide/header-map to docplanner/header-map across go.mod, .traefik.yml, docker-compose.yml, traefik.yaml, and README.md.
  • Upgrade golangci-lint config: Migrate .golangci.yml to v2 format, enable default: all with a refined disable list, and add formatters (gci, gofmt, gofumpt, goimports).
  • Fix all lint issues (26 total):
    • revive/var-naming: Rename Go package from header_map to headermap (no underscores).
    • err113: Replace dynamic fmt.Errorf calls with wrapped static sentinel errors (errEmptyMappings, errMissingSourceHeader, errMissingTargetHeader).
    • canonicalheader: Use canonical HTTP header names in test assertions (X-App-Id, X-Ua-Device).
    • paralleltest: Add t.Parallel() to all test functions and table-driven subtests.
    • wsl_v5: Add required blank lines before if statements and assignments following different statement types.
  • Update LICENSE copyright from personal name to Docplanner.
  • Fix README: Correct Docker port references (8888 / 9090) to match docker-compose.yml.

Test plan

  • make lint passes with 0 issues
  • go test ./... passes (all tests use t.Parallel() and canonical headers)
  • Verify Traefik local dev setup still works with the renamed plugin path (docker compose up)

@rodrigoechaide rodrigoechaide requested a review from a team March 2, 2026 17:33
@rodrigoechaide rodrigoechaide self-assigned this Mar 2, 2026
@rodrigoechaide rodrigoechaide changed the title Update header-map module references to use Docplanner namespace chore: update header-map module references to use docplanner namespace Mar 2, 2026
@rodrigoechaide rodrigoechaide changed the title chore: update header-map module references to use docplanner namespace chore: update header-map module references to use docplanner namespace CORE-384 Mar 2, 2026
Update Go module path and all Traefik plugin references from
rodrigoechaide/header-map to docplanner/header-map. Align docker
compose volumes, traefik config, and README examples with the new
namespace. Also update error message prefixes to use the canonical
package name.
@ishioni
Copy link

ishioni commented Mar 3, 2026

If we have ci config, we should probably do some workflows to run that CI...

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