Skip to content

Replay VCPKG_FIND_PACKAGE_#344

Merged
vicroms merged 3 commits intoMicrosoftDocs:mainfrom
BillyONeal:replay-top-level-find-package
Apr 7, 2025
Merged

Replay VCPKG_FIND_PACKAGE_#344
vicroms merged 3 commits intoMicrosoftDocs:mainfrom
BillyONeal:replay-top-level-find-package

Conversation

@BillyONeal
Copy link
Collaborator

@prmerger-automator
Copy link

@BillyONeal : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit ab15c52:

✅ Validation status: passed

File Status Preview URL Details
vcpkg/users/buildsystems/cmake-integration.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@BillyONeal BillyONeal marked this pull request as ready for review March 7, 2025 22:21
Copy link
Collaborator

@vicroms vicroms left a comment

Choose a reason for hiding this comment

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

Minor nit, but LGTM.

I would like to see an example of this feature being used in the maintainer guide as a tip to disable optional dependencies.

@learn-build-service-prod

This comment was marked as outdated.

@BillyONeal
Copy link
Collaborator Author

@vicroms Looks like that link is broken

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit c020edd:

✅ Validation status: passed

File Status Preview URL Details
vcpkg/users/buildsystems/cmake-integration.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@vicroms vicroms merged commit 406481a into MicrosoftDocs:main Apr 7, 2025
2 checks passed
@BillyONeal BillyONeal deleted the replay-top-level-find-package branch April 7, 2025 21:25
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.

3 participants