Skip to content

Docs refinements - #14591

Merged
Nir-Az merged 7 commits into
developmentfrom
update-readme
Jan 21, 2026
Merged

Nir-Az merged 7 commits into
developmentfrom
update-readme

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Jan 13, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings January 13, 2026 08:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the documentation for Windows installation requirements by revising the minimum CMake version and Visual Studio version specifications.

Changes:

  • Updated minimum CMake version requirement from v3.8+ to v3.10+
  • Changed Visual Studio version specification from listing specific versions (2017/2019/2022) to a more flexible >= 2017 format

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Nir-Az
Nir-Az requested a review from OhadMeir January 20, 2026 19:28
Comment thread doc/installation_windows.md Outdated
**Note:** Due to the USB 3.0 translation layer between native hardware and virtual machine, the librealsense team does not recommend or support installation in a VM.

librealsense shall be built on Windows using [CMake v3.8+](https://cmake.org/download/) and Visual Studio 2017 / 2019 / 2022:
librealsense shall be built on Windows using [CMake v3.10+](https://cmake.org/download/) and Visual Studio >= 2017:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think minimum supported VS is 2019

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks updated

@Nir-Az Nir-Az changed the title docs refinements Docs refinements Jan 21, 2026
@Nir-Az
Nir-Az merged commit 0a61068 into development Jan 21, 2026
53 of 54 checks passed
@Nir-Az
Nir-Az deleted the update-readme branch January 21, 2026 10:32
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.

3 participants