Repository navigation
release: 0.55.0 #82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
release: 0.55.0 #82
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| { | ||
| ".": "0.54.0" | ||
| ".": "0.55.0" | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| # File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details. | ||
|
|
||
| __title__ = "sent_dm" | ||
| __version__ = "0.54.0" # x-release-please-version | ||
| __version__ = "0.55.0" # x-release-please-version |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔴 Customers upgrading to 0.55.0 get a BREAKING CHANGES note that doesn't say what broke or what to do. The only line is "sync OpenAPI spec from production", an internal cause. CLAUDE.md asks for subjects that state "what changed, and what to do about it. Never an internal cause." This release removes
SMSMarket.sender_valuewith no alias, so code reading it breaks with no warning. Fix: before merging, rewrite both 0.55.0 lines (CHANGELOG.md:8 and :12) and the release PR body. Name the removedsender_valuefield, its replacement innumbers, and the new payment-method requirement. ABEGIN_COMMIT_OVERRIDEin the PR body is one way to do it.Why this was flagged
The changelog text is copied from the subject of commit a492eb1,
feat(api)!: sync OpenAPI spec from production. CHANGELOG.md:8 and :12 both repeat that subject, so the BREAKING CHANGES section names no field, method or behaviour. Commit a492eb1 deletessender_value: Optional[str]from src/sent_dm/types/channels/sms_market.py with no alias. A customer who readsmarket.sender_valuegets an AttributeError after upgrading, and the release notes give no hint of the cause or the replacement (numbers). The same commit adds a payment-method requirement for call participants (payment_method_missingin src/sent_dm/types/voice/call.py), and the notes don't mention it either. Unlike the 0.54.0 entry, which spells out what changed, this one tells customers nothing they can act on. CLAUDE.md marks CHANGELOG.md as owned by release-please, but that only means rewriting it in a release PR is expected. It does not exempt the text from the rule about customer-readable subjects. I checked this by reading the diff and the commit; nothing was run.Verification: CHANGELOG.md:8 (under "### ⚠ BREAKING CHANGES") and CHANGELOG.md:12 both say only "api: sync OpenAPI spec from production". The text names an internal process, not a customer-facing change.
git show a492eb1 -- src/sent_dm/types/channels/sms_market.pydeletessender_value: Optional[str] = NonefromSMSMarket, and no alias remains.