-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[PM-12989] Create process for qa to build client with particular sdk version #11601
[PM-12989] Create process for qa to build client with particular sdk version #11601
Conversation
…Client$` function
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #11601 +/- ##
==========================================
- Coverage 33.48% 33.47% -0.01%
==========================================
Files 2851 2854 +3
Lines 89283 89335 +52
Branches 17008 17012 +4
==========================================
+ Hits 29900 29909 +9
- Misses 57029 57071 +42
- Partials 2354 2355 +1 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
New Issues
Fixed Issues
|
0993fe9
to
d07fe7c
Compare
d07fe7c
to
276da3e
Compare
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.
👍🏻 Looking good!
🌱 Thank you for moving the error handling into the service layer!
There's a nit, but it's cool if we circle back. I don't think it's worth blocking this PR over. :)
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.
It can be for the future but I love the idea of having VersionService
available with the exact same interface for all the clients and then deprecating the method from PlatformUtilsService
. I want to slim down what is offered from PlatformUtilsService
.
…nt-with-particular-sdk-version
…nt-with-particular-sdk-version
…nt-with-particular-sdk-version
…nt-with-particular-sdk-version
🎟️ Tracking
Depends on bitwarden/sdk-internal#5
📔 Objective
This PR adds two things:
SDK version string
All clients now provide some way of displaying the underlying SDK version
Help -> About Bitwarden
Settings -> About -> About Bitwarden
bw --version
await __version()
in the consoleCI version override
Users can now trigger manual CI build with an optional SDK branch override. This override will download the latest successful SDK build for that branch and use it to build the client.
📸 Screenshots
⏰ Reminders before review
🦮 Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or ℹ️ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or 💭 (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or:warning:
) for more significant problems or concerns needing attention:seedling:
) or ♻️ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes