From c0503dedd1da4722b51a4deb2bf1f1451d3ca64b Mon Sep 17 00:00:00 2001 From: kinchit Date: Tue, 27 Oct 2026 16:20:00 +0530 Subject: [PATCH] chore: stop publishing the Go package smallcase/cdk-vpc-module-go is archived, so the Go publish job fails on every release. Because the GitHub release job needs all publishers to succeed, no git tag or GitHub release is created, projen recomputes the same version each time, and npm rejects the duplicate (0.0.23). That blocks every new release, including skipDefaultNatRoute (#34). Remove publishToGo and regenerate the projen files (release/build workflows, package:go task, mergify check). Co-Authored-By: Claude Sonnet 5.5 --- .github/workflows/build.yml | 47 ------------------------------ .github/workflows/release.yml | 55 ----------------------------------- .mergify.yml | 2 -- .projen/tasks.json | 17 ----------- .projenrc.js | 5 ---- package.json | 4 --- 6 files changed, 130 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 3ebc78f..21c1d0d 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -184,50 +184,3 @@ jobs: - name: Collect python artifact id: collect_python_artifact run: mv .repo/dist dist - package-go: - needs: build - runs-on: ubuntu-latest - permissions: - contents: read - if: ${{ !needs.build.outputs.self_mutation_happened }} - steps: - - uses: actions/setup-node@v6 - with: - node-version: lts/* - package-manager-cache: false - - uses: actions/setup-go@v6 - with: - go-version: ^1.18.0 - cache: false - - name: Download build artifacts - id: download_build_artifacts - uses: actions/download-artifact@v8 - with: - artifact-ids: ${{ needs.build.outputs.artifact_id }} - path: dist - - name: Restore build artifact permissions - id: restore_build_artifact_permissions - run: cd dist && setfacl --restore=permissions-backup.acl - continue-on-error: true - - name: Checkout - id: checkout - uses: actions/checkout@v6 - with: - ref: ${{ github.event.pull_request.head.ref }} - repository: ${{ github.event.pull_request.head.repo.full_name }} - path: .repo - - name: Install Dependencies - id: install_dependencies - run: cd .repo && yarn install --check-files --frozen-lockfile - - name: Extract build artifact - id: extract_build_artifact - run: tar --strip-components=1 -xzvf dist/js/*.tgz -C .repo - - name: Move build artifact out of the way - id: move_build_artifact_out_of_the_way - run: mv dist dist.old - - name: Create go artifact - id: create_go_artifact - run: cd .repo && npx projen package:go - - name: Collect go artifact - id: collect_go_artifact - run: mv .repo/dist dist diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 6a28b40..c7f6b8c 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -82,7 +82,6 @@ jobs: - release - release_npm - release_pypi - - release_golang runs-on: ubuntu-latest permissions: contents: write @@ -212,57 +211,3 @@ jobs: TWINE_PASSWORD: ${{ secrets.TWINE_PASSWORD }} PUBLIB_DRYRUN: ${{ inputs.dry_run }} run: npx -p publib@latest publib-pypi - release_golang: - name: Publish to GitHub Go Module Repository - needs: release - runs-on: ubuntu-latest - permissions: - contents: read - if: needs.release.outputs.tag_exists != 'true' && needs.release.outputs.latest_commit == github.sha - steps: - - uses: actions/setup-node@v6 - with: - node-version: lts/* - package-manager-cache: false - - uses: actions/setup-go@v6 - with: - go-version: ^1.18.0 - cache: false - - name: Download build artifacts - id: download_build_artifacts - uses: actions/download-artifact@v8 - with: - artifact-ids: ${{ needs.release.outputs.artifact_id }} - path: dist - - name: Restore build artifact permissions - id: restore_build_artifact_permissions - run: cd dist && setfacl --restore=permissions-backup.acl - continue-on-error: true - - name: Checkout - id: checkout - uses: actions/checkout@v6 - with: - path: .repo - - name: Install Dependencies - id: install_dependencies - run: cd .repo && yarn install --check-files --frozen-lockfile - - name: Extract build artifact - id: extract_build_artifact - run: tar --strip-components=1 -xzvf dist/js/*.tgz -C .repo - - name: Move build artifact out of the way - id: move_build_artifact_out_of_the_way - run: mv dist dist.old - - name: Create go artifact - id: create_go_artifact - run: cd .repo && npx projen package:go - - name: Collect go artifact - id: collect_go_artifact - run: mv .repo/dist dist - - name: Release - id: release - env: - GIT_USER_NAME: sc-infra-bot - GIT_USER_EMAIL: infra@smallcase.com - GITHUB_TOKEN: ${{ secrets.GO_GITHUB_TOKEN }} - PUBLIB_DRYRUN: ${{ inputs.dry_run }} - run: npx -p publib@latest publib-golang diff --git a/.mergify.yml b/.mergify.yml index 49d9d98..7f22a90 100644 --- a/.mergify.yml +++ b/.mergify.yml @@ -9,7 +9,6 @@ queue_rules: - status-success=build - status-success=package-js - status-success=package-python - - status-success=package-go merge_method: squash commit_message_format: title: pr-title @@ -25,6 +24,5 @@ pull_request_rules: - status-success=build - status-success=package-js - status-success=package-python - - status-success=package-go merge_queue: max_parallel_checks: 1 diff --git a/.projen/tasks.json b/.projen/tasks.json index 333eb45..fc1fcce 100644 --- a/.projen/tasks.json +++ b/.projen/tasks.json @@ -210,23 +210,6 @@ }, { "spawn": "package:python" - }, - { - "spawn": "package:go" - } - ] - }, - "package:go": { - "name": "package:go", - "description": "Create go language bindings", - "steps": [ - { - "execArgs": [ - "jsii-pacmak", - "-v", - "--target", - "go" - ] } ] }, diff --git a/.projenrc.js b/.projenrc.js index 62c80e7..2ecbf12 100644 --- a/.projenrc.js +++ b/.projenrc.js @@ -26,11 +26,6 @@ const project = new AwsCdkConstructLibrary({ distName: 'cdk-vpc-module', module: 'cdk_vpc_module', }, - publishToGo: { - gitUserName: 'sc-infra-bot', - gitUserEmail: 'infra@smallcase.com', - moduleName: 'github.com/smallcase/cdk-vpc-module-go', - }, releaseEveryCommit: true, licensed: true, /* Indicates if a license should be added. */ dependabot: false, /* Include dependabot configuration. */ diff --git a/package.json b/package.json index 14043b1..91d2ea1 100644 --- a/package.json +++ b/package.json @@ -17,7 +17,6 @@ "eslint": "projen eslint", "package": "projen package", "package-all": "projen package-all", - "package:go": "projen package:go", "package:js": "projen package:js", "package:python": "projen package:python", "post-compile": "projen post-compile", @@ -121,9 +120,6 @@ "python": { "distName": "cdk-vpc-module", "module": "cdk_vpc_module" - }, - "go": { - "moduleName": "github.com/smallcase/cdk-vpc-module-go" } }, "tsconfig": "tsconfig.json",