Reference Desktop Native Rust deps install scripts#791
Conversation
|
Great job! No new security vulnerabilities introduced in this pull request |
There was a problem hiding this comment.
I hesitate to put it in the general Tools page, because that is not specific for clients
There is a spot for this: see docs/getting-started/clients/index.md > Requirements heading. That links out to the general Tools page today, but there's no reason that we can't have information directly under that heading. To me this entire Dependencies section could/should be on that page given that it's part of the monorepo tooling.
I'm definitely open to alternative locations for the information. I do think though that the details of the Rust tooling makes sense to be homed in the Desktop Native page, because it's only used in practice there. What do you think about the following: similar to how clients/index.md |
|
That sounds good to me! |
Deploying contributing-docs with
|
| Latest commit: |
6075419
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3451786e.contributing-docs.pages.dev |
| Branch Preview URL: | https://dn-pm-34175-reference-deskto.contributing-docs.pages.dev |
eliykat
left a comment
There was a problem hiding this comment.
Thanks for making all these changes to improve our DX!

🎟️ Tracking
https://bitwarden.atlassian.net/browse/PM-34175
📔 Objective
Reference the new Rust env install script in the DN dependencies section.
I hesitate to put it in the general Tools page, because that is not specific for
clients, and the other repos in the project might have different versioning requirements for the cargo binaries or toolchains.📸 Screenshots