Skip to content

Fix TriggerTagBot to handle multi-package registration PRs#365

Open
timholy wants to merge 1 commit into
masterfrom
teh/fix-trigger-tagbot-multi-package
Open

Fix TriggerTagBot to handle multi-package registration PRs#365
timholy wants to merge 1 commit into
masterfrom
teh/fix-trigger-tagbot-multi-package

Conversation

@timholy
Copy link
Copy Markdown
Member

@timholy timholy commented May 23, 2026

A registration PR that updates multiple packages' Versions.toml files (e.g. #352, which registered TileTrees 1.4 and TiledFactorizations 1.0 together) made PACKAGENAME a multiline value. Writing a multiline string to $GITHUB_ENV without heredoc syntax fails with Invalid format, so the step errored out and no repository_dispatch was sent — leaving TiledFactorizations v1.0.0 untagged.

Loop over package names within a single step instead, and dispatch once per package.

A registration PR that updates multiple packages' Versions.toml files
(e.g. #352, which registered TileTrees 1.4 and
TiledFactorizations 1.0 together) made `PACKAGENAME` a multiline value.
Writing a multiline string to $GITHUB_ENV without heredoc syntax fails
with `Invalid format`, so the step errored out and no `repository_dispatch`
was sent — leaving TiledFactorizations v1.0.0 untagged.

Loop over package names within a single step instead, and dispatch once
per package.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@timholy
Copy link
Copy Markdown
Member Author

timholy commented May 23, 2026

CC @IanButterworth

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