Use stricter types for `CommitFunctions` and `ChangelogFunctions` by bluwy · Pull Request #2040 · changesets/changesets · GitHub
Skip to content

Use stricter types for CommitFunctions and ChangelogFunctions - #2040

Merged
bluwy merged 4 commits into
nextfrom
strict-function-types
May 28, 2026
Merged

bluwy merged 4 commits into
nextfrom
strict-function-types

Conversation

@bluwy

@bluwy bluwy commented May 27, 2026

Copy link
Copy Markdown
Member

fix #1722

also in the valibot refactor the opts is also validated as Record<string, unknown>, so this matches it.

@changeset-bot

changeset-bot Bot commented May 27, 2026 •

Copy link
Copy Markdown

@codecov

codecov Bot commented May 27, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.59%. Comparing base (61eef67) to head (4eda5bb).

Files with missing lines Patch % Lines
packages/changelog-github/src/index.ts 60.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             next    #2040      +/-   ##
==========================================
- Coverage   80.62%   80.59%   -0.03%     
==========================================
  Files          61       61              
  Lines        2317     2319       +2     
  Branches      636      636              
==========================================
+ Hits         1868     1869       +1     
- Misses        397      398       +1     
  Partials       52       52              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Those are validated by valibot at schema-check time now right?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yes, this aligns the type with the Valibot schemas

Comment thread packages/types/src/index.ts
@bluwy
bluwy added this pull request to the merge queue May 28, 2026
Merged via the queue into next with commit 88f2abb May 28, 2026
10 of 12 checks passed
@bluwy
bluwy deleted the strict-function-types branch May 28, 2026 06:50
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