Skip to content

Add changeset requirement to package guidelines#54

Merged
jln13x merged 1 commit intomainfrom
jln13x/add-changeset-docs
Jan 24, 2026
Merged

Add changeset requirement to package guidelines#54
jln13x merged 1 commit intomainfrom
jln13x/add-changeset-docs

Conversation

@jln13x
Copy link
Owner

@jln13x jln13x commented Jan 24, 2026

Document that package changes require a changeset generated via pnpm changeset with patch version.

This ensures all package modifications have corresponding version bump records.

🤖 Generated with Claude Code

Document that package changes require a changeset generated via `pnpm changeset` with patch version.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Jan 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
jlnstack Ready Ready Preview, Comment Jan 24, 2026 9:25am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 24, 2026

Greptile Overview

Greptile Summary

Added a new "Changesets" section under Package Guidelines documenting the requirement for package changes to include a changeset. The addition specifies running pnpm changeset, selecting affected packages, and choosing patch version.

  • Simple documentation addition that aligns with the repository's existing changeset workflow (confirmed by .changeset/config.json and package.json scripts)
  • Well-placed within the Package Guidelines section after type and export guidance
  • Clear and concise instructions for contributors

Confidence Score: 5/5

  • This PR is completely safe to merge with no risk
  • This is a documentation-only change that adds a clear guideline about changesets. The instruction is accurate (verified against the repository's changeset configuration), well-formatted, and appropriately placed in the document structure. No code changes, no potential bugs, and follows the existing documentation style.
  • No files require special attention

@jln13x jln13x merged commit 47f693f into main Jan 24, 2026
5 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.

1 participant