Skip to content

Fix GitHub release repository context - #28

Merged
andymeierdev merged 1 commit into
mainfrom
andymeierdev/fix-publish-release-context
Aug 7, 2026
Merged

Fix GitHub release repository context#28
andymeierdev merged 1 commit into
mainfrom
andymeierdev/fix-publish-release-context

Conversation

@andymeierdev

Copy link
Copy Markdown
Collaborator

Summary

  • pass the explicit GitHub repository to both release lookup and creation
  • preserve idempotent release retries without requiring a repository checkout

Root cause

The Publish job downloads the verified package but does not check out the repository. GitHub CLI therefore failed with fatal: not a git repository while creating v2026.8.0.

Verification

  • reproduced the failure from a directory outside a Git checkout
  • confirmed gh release view succeeds there with --repo meiermade/FSharp.ViewEngine
  • confirmed the existing release is detected by the updated idempotent shell logic
  • validated publish.yml with actionlint 1.7.12
  • git diff --check

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

🍹 preview on fsharp-view-engine/prod

Pulumi report

View in Pulumi Cloud

  Previewing update (prod)

View Live: https://app.pulumi.com/meiermade/fsharp-view-engine/prod/previews/eb338957-9ca6-4bd8-9adb-f235abdb8890

pulumi:pulumi:Stack: (same)
  [urn=urn:pulumi:prod::fsharp-view-engine::pulumi:pulumi:Stack::fsharp-view-engine-prod]
  ~ kubernetes:apps/v1:Deployment: (update)
      [id=fsharpviewengine/fsharpviewengine]
      [urn=urn:pulumi:prod::fsharp-view-engine::kubernetes:apps/v1:Deployment::fsharpviewengine]
    ~ spec: {
        ~ template: {
            ~ spec: {
                ~ containers: [
                    ~ [0]: {
                            ~ env: [
                                ~ [0]: {
                                        ~ value: "2026.8.0" => "preview"
                                      }
                                ~ [1]: {
                                        ~ value: "f20feaf5a823881624e57cce338e860c0e02bfd7" => "702287cc3be91e5230f27911027a6e5f2dccef7c"
                                      }
                              ]
                          }
                  ]
              }
          }
      }
Resources:
  ~ 1 to update
  9 unchanged
  

@andymeierdev
andymeierdev merged commit 207b4be into main Aug 7, 2026
3 checks passed
@andymeierdev
andymeierdev deleted the andymeierdev/fix-publish-release-context branch August 7, 2026 11:39
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.

1 participant