Skip to content
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

[ftxui] update to 2.0.0 #22955

Merged
merged 1 commit into from
Feb 8, 2022
Merged

Conversation

ArthurSonzogni
Copy link
Contributor

Describe the pull request

  • What does your PR fix?

    • Add version 2.0.0
    • Use version-semver instead of version-date
  • Which triplets are supported/not supported? Have you updated the CI baseline?

    Similar to [ftxui] update to 0.11 #22293, uwp is not supported. I haven't touched ci.baseline.txt. Should I? Not clear to me.

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@ghost
Copy link

ghost commented Feb 5, 2022

CLA assistant check
All CLA requirements met.

@JackBoosY JackBoosY self-assigned this Feb 6, 2022
@JackBoosY JackBoosY added the category:port-update The issue is with a library, which is requesting update new revision label Feb 6, 2022
ports/ftxui/vcpkg.json Outdated Show resolved Hide resolved
- Switch to version-semver
- Add version 2.0.0. Released 14 days ago:
https://github.com/ArthurSonzogni/FTXUI/releases/tag/v2.0.0
@@ -1,6 +1,6 @@
{
"name": "ftxui",
"version-date": "2021-10-17",
"version-semver": "2.0.0",
Copy link
Contributor

@JackBoosY JackBoosY Feb 7, 2022

Choose a reason for hiding this comment

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

Note to other reviewers: See ArthurSonzogni/FTXUI#207.

@JackBoosY JackBoosY added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Feb 7, 2022
Copy link
Contributor

@ras0219-msft ras0219-msft left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the PR!

@@ -1,6 +1,6 @@
{
"name": "ftxui",
"version-date": "2021-10-17",
"version-semver": "2.0.0",
"port-version": 1,
Copy link
Contributor

Choose a reason for hiding this comment

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

Conventionally we remove the port-version (resetting it to 0) whenever the main version updates. However, this isn't a significant enough issue to warrant blocking the PR.

@ras0219-msft ras0219-msft merged commit 0573bca into microsoft:master Feb 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants