Skip to content

feat: fix how needs_bootstrap works to match what clients need#856

Merged
djsmith85 merged 1 commit intomainfrom
coroiu/tweak-needs-bootstrap
Mar 18, 2026
Merged

feat: fix how needs_bootstrap works to match what clients need#856
djsmith85 merged 1 commit intomainfrom
coroiu/tweak-needs-bootstrap

Conversation

@coroiu
Copy link
Contributor

@coroiu coroiu commented Mar 18, 2026

🎟️ Tracking

📔 Objective

Required fix for Daniels PR

🚨 Breaking Changes

@coroiu coroiu requested a review from a team as a code owner March 18, 2026 13:51
@coroiu coroiu requested a review from djsmith85 March 18, 2026 13:51
@github-actions
Copy link
Contributor

Logo
Checkmarx One – Scan Summary & Detailsff651a3c-0ba0-4e7f-99fc-80b4aa7b54be

Great job! No new security vulnerabilities introduced in this pull request

@sonarqubecloud
Copy link

@djsmith85 djsmith85 enabled auto-merge (squash) March 18, 2026 13:58
@github-actions
Copy link
Contributor

github-actions bot commented Mar 18, 2026

🔍 SDK Breaking Change Detection Results

SDK Version: coroiu/tweak-needs-bootstrap (a37f74b)
Completed: 2026-03-18 14:04:10 UTC
Total Time: 308s

Client Status Details
typescript ❌ Breaking changes detected TypeScript compilation failed with new SDK version - View Details

Breaking change detection completed. View SDK workflow

@djsmith85 djsmith85 merged commit 9db98d4 into main Mar 18, 2026
52 of 56 checks passed
@djsmith85 djsmith85 deleted the coroiu/tweak-needs-bootstrap branch March 18, 2026 14:09
bw-ghapp bot pushed a commit to bitwarden/sdk-swift that referenced this pull request Mar 18, 2026
djsmith85 added a commit that referenced this pull request Mar 18, 2026
djsmith85 added a commit that referenced this pull request Mar 18, 2026
## 📔 Objective

<!-- Describe what the purpose of this PR is, for example what bug
you're fixing or new feature you're adding. -->
Fixes failing tests caused by
#856

## 🚨 Breaking Changes

<!-- Does this PR introduce any breaking changes? If so, please describe
the impact and migration path for clients.

If you're unsure, the automated TypeScript compatibility check will run
when you open/update this PR and provide feedback.

For breaking changes:
1. Describe what changed in the client interface
2. Explain why the change was necessary
3. Provide migration steps for client developers
4. Link to any paired client PRs if needed

Otherwise, you can remove this section. -->

Co-authored-by: Daniel James Smith <djsmith85@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants