Skip to content

Rename people_and_teams category to people#114

Merged
invisiblemask merged 2 commits intomainfrom
rename-people-and-teams-category
Mar 30, 2026
Merged

Rename people_and_teams category to people#114
invisiblemask merged 2 commits intomainfrom
rename-people-and-teams-category

Conversation

@invisiblemask
Copy link
Copy Markdown
Contributor

Summary

  • Renames the people_and_teams category enum value to people
  • Adds changeset for minor version bump

Test plan

  • Verify build passes
  • Confirm changeset is picked up correctly

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 30, 2026

🦋 Changeset detected

Latest commit: a61ce8d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@everipedia/iq-utils Major

Not sure what this means? Click here to learn what changesets are.

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

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request renames the "people_and_teams" category to "people" within the Category enum. A review comment correctly identifies that this modification is a breaking API change and suggests updating the changeset version bump from minor to major to adhere to semantic versioning.

Comment thread .changeset/rename-people-category.md Outdated
@@ -0,0 +1,5 @@
---
"@everipedia/iq-utils": minor
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Renaming an enum value (people_and_teams to people) is a breaking API change. According to Semantic Versioning, this should trigger a major version bump to signal an incompatible API change to consumers of the library. An incorrect version bump can cause issues in downstream projects that rely on this package. Please update this to major.

Suggested change
"@everipedia/iq-utils": minor
"@everipedia/iq-utils": major

@invisiblemask invisiblemask merged commit ef8fe4b into main Mar 30, 2026
1 check 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