Skip to content

add installation of gen-commands - #10

Merged
gregmankes merged 4 commits into
developfrom
gmankes/update_gen_commands_rule
Feb 6, 2026
Merged

add installation of gen-commands#10
gregmankes merged 4 commits into
developfrom
gmankes/update_gen_commands_rule

Conversation

@gregmankes

@gregmankes gregmankes commented Feb 4, 2026

Copy link
Copy Markdown

What was changed

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Note

Low Risk
Build tooling changes only; main risk is CI/local build breakage due to relying on git clone/network availability and the upstream temporalio/cli repo layout.

Overview
Makes code generation self-contained by adding a Makefile install target that temporarily clones temporalio/cli and go installs cmd/gen-commands, and making gen depend on it.

Removes the go.mod tool pin for gen-commands and trims now-unused module dependencies, and updates .gitignore to exclude the temporary cli/ clone directory.

Written by Cursor Bugbot for commit 0019b0c. This will update automatically on new commits. Configure here.

@gregmankes
gregmankes requested a review from a team as a code owner February 4, 2026 20:20
Comment thread Makefile
Comment thread Makefile
Comment thread Makefile

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Comment thread .gitignore Outdated
@gregmankes
gregmankes merged commit 37ef708 into develop Feb 6, 2026
3 checks passed
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