Skip to content

Tag published version and run check-types if declared.#2

Open
pkaminski wants to merge 1 commit into
masterfrom
tag-version
Open

Tag published version and run check-types if declared.#2
pkaminski wants to merge 1 commit into
masterfrom
tag-version

Conversation

@pkaminski
Copy link
Copy Markdown
Member

@pkaminski pkaminski commented May 15, 2026

This will also require switching to write permissions in all callers; I'll commit those changes directly.


This change is Reviewable

This will also require switching to write permissions in all callers;
I'll commit those changes directly.
@pkaminski pkaminski requested a review from snoack May 15, 2026 22:53
Copy link
Copy Markdown

@snoack snoack left a comment

Choose a reason for hiding this comment

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

@snoack reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on pkaminski).


.github/workflows/publish.yaml line 184 at r1 (raw file):

          const name = pkg.name;
          const version = pkg.version;
          fs.appendFileSync(process.env.GITHUB_OUTPUT, `package_name=${name}\n`);

Does the package name even matter? In practice it's just the name of the repo, and therefore redundant in the commit message, or am I missing something?

Copy link
Copy Markdown
Member Author

@pkaminski pkaminski left a comment

Choose a reason for hiding this comment

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

@pkaminski made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on snoack).


.github/workflows/publish.yaml line 184 at r1 (raw file):

Previously, snoack (Sebastian Noack) wrote…

Does the package name even matter? In practice it's just the name of the repo, and therefore redundant in the commit message, or am I missing something?

Doesn't terribly matter, but Codex decided to give the tags' messages the full name@version treatment (see next to last line), and given that it's just a couple extra lines here I find it hard to get upset about it.

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