Skip to content

[vcpkg] x-add-version now also checks if the manifest file is properly formatted#43

Merged
vicroms merged 3 commits intomicrosoft:mainfrom
autoantwort:error-when-not-formatted
Apr 12, 2021
Merged

[vcpkg] x-add-version now also checks if the manifest file is properly formatted#43
vicroms merged 3 commits intomicrosoft:mainfrom
autoantwort:error-when-not-formatted

Conversation

@autoantwort
Copy link
Copy Markdown
Contributor

@strega-nil
Copy link
Copy Markdown
Contributor

I think this is a good idea, although it needs a --skip-manifest-checks or something flag.

@vicroms vicroms added the requires:discussion This PR requires discussion of the correct way forward label Apr 7, 2021
@vicroms
Copy link
Copy Markdown
Member

vicroms commented Apr 7, 2021

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@autoantwort
Copy link
Copy Markdown
Contributor Author

Are the tests using the latest master with microsoft/vcpkg#17120 ?

@autoantwort autoantwort force-pushed the error-when-not-formatted branch from c93efaf to c285fd8 Compare April 8, 2021 12:14
@autoantwort
Copy link
Copy Markdown
Contributor Author

Are the tests using the latest master with microsoft/vcpkg#17120 ?

Or should I add --skip-formatting-check here

@autoantwort autoantwort force-pushed the error-when-not-formatted branch from 2ba724c to 3b30b72 Compare April 8, 2021 13:59
@autoantwort autoantwort force-pushed the error-when-not-formatted branch 2 times, most recently from 28fe72a to 8278f8a Compare April 8, 2021 18:31
@autoantwort
Copy link
Copy Markdown
Contributor Author

This is really needed. Especially now when the PR template asks if you have run x-add-version

@vicroms vicroms merged commit bb3d813 into microsoft:main Apr 12, 2021
strega-nil added a commit that referenced this pull request Apr 15, 2021
The following PRs are included:

* hopefully fix crash in constraints (#60)
* [vcpkg] allow --version to check the version (#50)
* Remove baseline warning (#27)
* [git] always pass autocrlf=false (#58)
* ignore QtCreator CMake project files (#54)
* ignore .DS_store files (#53)
* [vcpkg] x-add-version now also checks if the manifest file is properly formatted (#43)
* hopefully fix ci issue #16773 (#34)
* Add docs to set VCPKG_ROOT to run tests (#45)
* [vcpkg] x-add-version improve speed by calling get_builtin_baseline only once (#44)
* add clang-format version to format-cxxcode (#41)
* [vcpkg] Introduce experimental workaround X_VCPKG_NUGET_ID_PREFIX (#40)
* [supports] Add `native` identifier expression and x-check-support command (#29)
* [metrics] Split reporting of installs into name:triplet (#39)
* [vcpkg] Improve error when accessing missing feature (#38)
* [vcpkg] Allow shallow git registries (#37)
* Disable git autocrlf when archiving tree (#36)
* Use only named packages from extra registries (#35)
* [registries] add metrics (#30)
* Add vcpkg policy cmake helper port support (#17)
* [osx] add support for rosetta (#23)
* don't build tls12-download unless it's needed (#33)
* Add new telemetry points for versioning (#21)
* add cmake_minimum_required to vcpkg_tags (#25)
* [x-add-versions] Perform atomic replacement of versioning files (#28)
* [tools] support gsutil (#19)
* add CUDA 11.1 and 11.2 to KEEP_ENV_VARS defaults (#26)
* Add finite timeout on CURL metrics endpoint. (#22)
* fix UB in make_error_code(utf8_errc) (#18)
@autoantwort autoantwort deleted the error-when-not-formatted branch September 4, 2021 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires:discussion This PR requires discussion of the correct way forward

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check format of vcpkg.json file in x-add-version

3 participants