Skip to content

Comments

Improve publish workflow comment with actual package versions#6380

Open
kulesy wants to merge 1 commit intomainfrom
claude/fix-tinacms-publish-alias-53t4F
Open

Improve publish workflow comment with actual package versions#6380
kulesy wants to merge 1 commit intomainfrom
claude/fix-tinacms-publish-alias-53t4F

Conversation

@kulesy
Copy link
Contributor

@kulesy kulesy commented Jan 28, 2026

Description

Updates the GitHub Actions publish workflow to generate more accurate and helpful installation instructions in PR comments. Instead of using a generic tag placeholder, which was causing issues when trying to install as it thought I was pointing to a sub packages like package@foo/bar.

The PR comment for tagged releases was using the branch name as the npm
tag for install commands, which caused issues when users tried to install
(npm interpreted it as a directory path). Now uses the actual version
numbers from the published packages instead.

https://claude.ai/code/session_012rv6RM3g1B2nE2kZaNwmYx
@kulesy kulesy requested a review from a team as a code owner January 28, 2026 23:14
@changeset-bot
Copy link

changeset-bot bot commented Jan 28, 2026

⚠️ No Changeset found

Latest commit: 5b3dfee

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

Playwright E2E Test Results

passed  7 passed
skipped  2 skipped

Details

stats  9 tests across 6 suites
duration  32.2 seconds
commit  5b3dfee

Skipped tests

chromium › retain-input-value-from-localstorage.spec.ts › Local storage retain edit test
chromium › rich-text-editor/newline-parsing.spec.ts › should be able to parse newline from markdown

console.log('📝 Generated packages section:', packagesSection);

// Generate install commands using actual versions
let installCommands = packages.map(pkg => `pnpm add ${pkg.name}@${pkg.version}`).join('\n');
Copy link
Member

Choose a reason for hiding this comment

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

do we expect this to run?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No this is just for the user to copy to run it themselves easily

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