-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
[scripts-audit] vcpkg_check_features #16192
Closed
strega-nil
wants to merge
2
commits into
microsoft:master
from
strega-nil:scripts-audit/vcpkg_check_features
Closed
[scripts-audit] vcpkg_check_features #16192
strega-nil
wants to merge
2
commits into
microsoft:master
from
strega-nil:scripts-audit/vcpkg_check_features
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
strega-nil
added
category:scripts-audit
Part of the scripts audit effort.
info:internal
This PR or Issue was filed by the vcpkg team.
labels
Feb 12, 2021
vicroms
approved these changes
Feb 12, 2021
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
The otl and ktx regressions was fixed. Please rerun the pipeline test. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
vicroms
approved these changes
Feb 23, 2021
* pull the cmake doc comment parsing out into its own function * support cmake helper ports * add real support for deprecation, as opposed to ad-hoc
* add a z_ in front of internal functions * move internal functions out set feature_vars again in parent scope
strega-nil
force-pushed
the
scripts-audit/vcpkg_check_features
branch
from
February 26, 2021 20:42
8561bfe
to
f311d87
Compare
Closing in favor of a single PR that contains all of these PRs. (to avoid destroying the CI system) |
strega-nil
added a commit
to strega-nil/vcpkg
that referenced
this pull request
Feb 26, 2021
* add a z_ in front of internal functions * move internal functions out set feature_vars again in parent scope
strega-nil
added a commit
that referenced
this pull request
Feb 28, 2021
* [scripts-audit rollup] PR #16419 * pull the cmake doc comment parsing out into its own function * support cmake helper ports * add real support for deprecation, as opposed to ad-hoc * [scripts-audit rollup] PR #16192 * add a z_ in front of internal functions * move internal functions out set feature_vars again in parent scope * [scripts-audit rollup] PR #16309 Audit vcpkg_copy_pdbs * [scripts-audit rollup] PR #16304 * Fix usage, documentation * [scripts-audit rollup] PR #16393 * [scripts-audit rollup] PR #16377 Deprecate `vcpkg_*_cmake` in favor of `vcpkg_cmake_*` from the `vcpkg-cmake` port, as well as `vcpkg_fixup_cmake_targets` in favor of `vcpkg_cmake_config_fixup` from the `vcpkg-cmake-config` port.
51 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
category:scripts-audit
Part of the scripts audit effort.
info:internal
This PR or Issue was filed by the vcpkg team.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See PR #16055, issue #16188
Part of the "merge 2021-02-26 EOD" group.