Fix install-tools

Open Beginner friendly
#2,736 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 2 days

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
64/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
go

Research direction

Start by reading the tools module lock file and the Makefile targets for install-tools, golint, and goporto. Reproduce the reported undefined goheader.Configuration error with make install-tools, then check that the affected targets run successfully with compatible go-header and golangci-lint versions.

Written by the indexing model from the issue text.

Description

go

since golangci-lint is kept: it doesn't build in the tools module right now. make install-tools on this branch fails with undefined: goheader.Configuration, because go-header was bumped to v1.0.1 by the lock file PRs while golangci-lint v2.15.0 expects v0.5.0. So make golint and make goporto still can't run. Not blocking, just flagging.

As per https://github.com/open-telemetry/opentelemetry-lambda/pull/2726#pullrequestreview-5824058957

Dominant language
Go
Stars
433
Forks
249
Avg merge
1d 12h
Merged PRs (30d)
39

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from open-telemetry/opentelemetry-lambda

All issues in open-telemetry/opentelemetry-lambda

Similar issues

More Go issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.