Skip to content

cdktf migration: upgrade go and some deps - #84

Open
filiphaftek wants to merge 1 commit into
cdk-terrain-migrationfrom
fh/cdktn_updates
Open

cdktf migration: upgrade go and some deps#84
filiphaftek wants to merge 1 commit into
cdk-terrain-migrationfrom
fh/cdktn_updates

Conversation

@filiphaftek

Copy link
Copy Markdown
Contributor

Summary

  • Bump github.com/sourcegraph/cdktf-provider-gen to the current main HEAD (1dd7bde10a62) — the previously pinned commit (ce2a4875e27f) no longer exists upstream (force-pushed/rebased away), which broke go get -u
  • Update transitive dependencies pulled in by the new commit's dependency graph (golang.org/x/*, hashicorp/hc-install, ProtonMail/go-crypto, etc.) and bump the go directive 1.25.0 → 1.26.2
  • Drop transitive deps no longer required by the updated generator (chroma, glamour, jackc/pgconn, sourcegraph/sourcegraph/lib, etc.)

Test plan

  • go build -o .bin/cdktf-provider-gen github.com/sourcegraph/cdktf-provider-gen/cmd/cdktf-provider-gen builds successfully

Tests

Tested e2e via:

  1. make install and make -j4
  2. used in controller via replace in go.mod
  3. regenerated mi2 generate kustomize and all was applied: https://app.terraform.io/app/sourcegraph/workspaces?search=src-95e0e0f227324da05333&sort=-latest-change-at

Comment thread go.mod
go 1.26.2

require github.com/sourcegraph/cdktf-provider-gen v0.0.0-20260902211826-ce2a4875e27f
require github.com/sourcegraph/cdktf-provider-gen v0.0.0-20260902230416-1dd7bde10a62

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ahh, this was pointing to a commit that no longer exist https://github.com/sourcegraph/cdktf-provider-gen/pull/10/commits because we merged to the PR.

thank you

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