Skip to content

build(deps): update Go and golangci-lint - #188

Open
dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
automation/go-toolchain
Open

dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
automation/go-toolchain

Conversation

@dd-octo-sts

@dd-octo-sts dd-octo-sts Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

What

Update Go and/or golangci-lint to their latest stable releases.
CI and release builds read these pins from go.mod and .golangci-lint-version.

Why

Pick up toolchain security fixes and keep the linter current.
Release sources: https://go.dev/dl/ and https://github.com/golangci/golangci-lint/releases.

E2E testing

The branch push triggers the full CI suite. Review its results before merging,
including linter compatibility with the new Go version.
Manual scenario: build with make build, verify the toolchain with
go version -m ./ddtest, then run ddtest plan in a configured test project
and confirm successful plan generation and Datadog requests.

@dd-octo-sts
dd-octo-sts Bot requested a review from a team as a code owner October 17, 2026 07:30

@datadog-datadog-prod-us1-2 datadog-datadog-prod-us1-2 Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bits Code Review: PASS

More details

No reportable defect was identified in the linter update. Compatibility remains unverified because the updated linter was not executed and CI status retrieval returned HTTP 403.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Bits Code Review · Commit 034a7cc

@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Pipelines  Tests  Code Coverage

✨ Unblock PR with BitsAI

❌ Errors

Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 1 Pipeline job failed

CI | lint — 🔧 Needs a code fix, caused by this PR

View more details · View in GitHub Actions

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 88.19% (-0.00%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 02248ca | Docs | View more details | Give us feedback!

@dd-octo-sts
dd-octo-sts Bot force-pushed the automation/go-toolchain branch 4 times, most recently from 1df1f3b to 51a824d Compare October 24, 2026 07:30
@dd-octo-sts
dd-octo-sts Bot force-pushed the automation/go-toolchain branch 2 times, most recently from c40b7d7 to 12dd981 Compare October 28, 2026 07:27
@dd-octo-sts
dd-octo-sts Bot force-pushed the automation/go-toolchain branch from 12dd981 to 02248ca Compare October 29, 2026 07:27
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.

0 participants