Skip to content

AFE-85: Pin dependencies (astro-cli)#2072

Open
jaketf wants to merge 5 commits intomainfrom
jakeferriero/afe-76-pinning-dependencies-temporarily
Open

AFE-85: Pin dependencies (astro-cli)#2072
jaketf wants to merge 5 commits intomainfrom
jakeferriero/afe-76-pinning-dependencies-temporarily

Conversation

@jaketf
Copy link
Copy Markdown
Contributor

@jaketf jaketf commented Apr 7, 2026

Description

  • pin actions with pinact (see example PR)

  • fix curl | sh in CircleCI and GitHub Actions workflows

  • update Dockerfiles with digest pins

  • ensure uv --frozen is used everywhere in CI builds

Motivation

AFE-85: Pinning dependencies temporarily

- add lint-actions-pins.yaml using astronomer/actions reusable workflow
- replace curl | bash goreleaser install with pinned binary download (v2.15.2)
- pin Dockerfile base image to SHA digest
@jaketf jaketf requested a review from a team as a code owner April 7, 2026 23:12
@coveralls-official
Copy link
Copy Markdown

coveralls-official bot commented Apr 7, 2026

Coverage Report for CI Build 0

Coverage remained the same at 39.386%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 63192
Covered Lines: 24889
Line Coverage: 39.39%
Coverage Strength: 9.51 hits per line

💛 - Coveralls

@jaketf jaketf marked this pull request as draft April 7, 2026 23:29
@jaketf jaketf changed the title AFE-76: Pin dependencies temporarily (astro-cli) AFE-85: Pin dependencies temporarily (astro-cli) Apr 7, 2026
@jaketf jaketf marked this pull request as ready for review April 8, 2026 22:02
astro-cli is a public repo and cannot call reusable workflows from
astronomer/actions which is internal. Inline the pinact-action steps
directly, mirroring what the upstream reusable workflow does.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jaketf jaketf changed the title AFE-85: Pin dependencies temporarily (astro-cli) AFE-85: Pin dependencies (astro-cli) Apr 8, 2026
name: goreleaser
command: curl -sL https://raw.githubusercontent.com/goreleaser/get/master/get | bash
command: |
GORELEASER_VERSION=v2.15.2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's use goreleaser/goreleaser-action instead.

Copy link
Copy Markdown
Contributor

@nickstenning nickstenning left a comment

Choose a reason for hiding this comment

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

A suggestion to use an existing action rather than our own script, but otherwise LGTM.

jaketf and others added 3 commits April 9, 2026 11:50
…(AFE-85)

Move the release job from CircleCI to a GitHub Actions workflow using
goreleaser/goreleaser-action. osxcross is set up on the GHA runner for
macOS CGO cross-compilation (CGO_ENABLED=1, CC=o64-clang) and cached
to avoid rebuilding on every run. The new-tag job in CircleCI still
creates and pushes the tag, which triggers the GHA release workflow.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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