Skip to content

feat: sidecar for TS and Python SDKs - #2

Open
mnafees wants to merge 15 commits into
mainfrom
nafees/sidecar
Open

feat: sidecar for TS and Python SDKs#2
mnafees wants to merge 15 commits into
mainfrom
nafees/sidecar

Conversation

@mnafees

@mnafees mnafees commented Aug 19, 2026

Copy link
Copy Markdown
Member

Description

Adds release mechanism for a brand new sidecar to run embedded inside TS and Python.

Related to hatchet-dev/hatchet#4742

Type of change

  • New feature (non-breaking change which adds functionality)
  • CI (any automation pipeline changes)

Checklist

Changes have been:

  • Tested (unit, integration, or manually with steps specified)
  • Linted and formatted
  • Documented (where applicable)

🤖 AI Disclosure
  • I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.
  • Details: [e.g. generating tests, writing docs]

@mnafees mnafees self-assigned this Aug 19, 2026
@mnafees
mnafees requested a balanced review from Copilot August 19, 2026 15:27

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread go.mod
gopkg.in/yaml.v3 v3.0.1 // indirect
)

replace github.com/hatchet-dev/hatchet => /Users/mnafees/git/hatchet-dev/hatchet-workstation/repos/hatchet

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

remove this?

Comment thread .goreleaser.yml
Comment on lines +39 to +50
- enabled: '{{ isEnvSet "MACOS_SIGN_P12" }}'
ids:
- hatchet-embedded-sidecar
sign:
certificate: "{{.Env.MACOS_SIGN_P12}}"
password: "{{.Env.MACOS_SIGN_PASSWORD}}"
notarize:
issuer_id: "{{.Env.MACOS_NOTARY_ISSUER_ID}}"
key_id: "{{.Env.MACOS_NOTARY_KEY_ID}}"
key: "{{.Env.MACOS_NOTARY_KEY}}"
wait: true
timeout: 20m

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

so these need to be set before we merge this?

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.

3 participants