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

[workflow] Update version to 0.9.6 #19280

Merged
merged 7 commits into from
Aug 5, 2021
Merged

Conversation

dengjun101
Copy link
Contributor

No description provided.

@dengjun101 dengjun101 marked this pull request as ready for review August 1, 2021 15:27
@JonLiu1993 JonLiu1993 self-assigned this Aug 2, 2021
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Aug 2, 2021
@JonLiu1993 JonLiu1993 changed the title update the port workflow version to 0.9.6 [workflow] Update version to 0.9.6 Aug 2, 2021
@@ -1,6 +1,6 @@
{
"name": "workflow",
"version": "0.9.5",
"version": "0.9.6",
"port-version": 1,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"port-version": 1,

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 261c458af6e3eed5d099144aff95d2b5035f656b -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/w-/workflow.json b/versions/w-/workflow.json
index 3adb15b..725cdd2 100644
--- a/versions/w-/workflow.json
+++ b/versions/w-/workflow.json
@@ -5,11 +5,6 @@
       "version": "0.9.6",
       "port-version": 0
     },
-    {
-      "git-tree": "b3f929dcfa8c14b9d70471c159dcd9a0ee3c0a8d",
-      "version": "0.9.6",
-      "port-version": 1
-    },
     {
       "git-tree": "3771249036075b8138d0e34de9799ca4571b1867",
       "version": "0.9.5",

Comment on lines 8 to 12
{
"git-tree": "b3f929dcfa8c14b9d70471c159dcd9a0ee3c0a8d",
"version": "0.9.6",
"port-version": 1
},
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{
"git-tree": "b3f929dcfa8c14b9d70471c159dcd9a0ee3c0a8d",
"version": "0.9.6",
"port-version": 1
},

Copy link
Member

Choose a reason for hiding this comment

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

We only need to add "version" once when submitting pr

This reverts commit 487548a.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 261c458af6e3eed5d099144aff95d2b5035f656b -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index aed4af1..895f9d5 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -6714,7 +6714,7 @@
     },
     "workflow": {
       "baseline": "0.9.6",
-      "port-version": 1
+      "port-version": 0
     },
     "wpilib": {
       "baseline": "2020.3.2",
diff --git a/versions/w-/workflow.json b/versions/w-/workflow.json
index 4eb4176..725cdd2 100644
--- a/versions/w-/workflow.json
+++ b/versions/w-/workflow.json
@@ -1,9 +1,9 @@
 {
   "versions": [
     {
-      "git-tree": "b3f929dcfa8c14b9d70471c159dcd9a0ee3c0a8d",
+      "git-tree": "8a8abc2f5725ad78500a5edd20df2c73ee2ddc37",
       "version": "0.9.6",
-      "port-version": 1
+      "port-version": 0
     },
     {
       "git-tree": "3771249036075b8138d0e34de9799ca4571b1867",

@JonLiu1993 JonLiu1993 added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Aug 3, 2021
@@ -2,16 +2,16 @@ if(VCPKG_TARGET_IS_WINDOWS AND NOT VCPKG_TARGET_IS_MINGW)
vcpkg_from_github(
OUT_SOURCE_PATH SOURCE_PATH
REPO sogou/workflow
REF 268c873d20b5cddedcf1d36fe9c2af8338353a98
SHA512 07b61657e34c1bce1f0fb8314e9d531e43dd9bf6004d6c67aaad9fcafa2073e05b835b85e87c82c26ca828edca08e9960a25578830657b9909b28bfa75cac6cc
REF e411d98144a56e2c2ffe4ee878f3b4e8a2cf9442
Copy link
Contributor

Choose a reason for hiding this comment

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

This commit seems much newer than 0.9.6 with several functional changes applied. We want to avoid cross-platform users being confused about things working on one platform but not working on another.

Is there a closer commit to v0.9.6 that's appropriate to use here instead? Perhaps sogou/workflow@b9919bc ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The core functions are consistent. It's fine. Thanks!

@JonLiu1993 JonLiu1993 added requires:author-response and removed info:reviewed Pull Request changes follow basic guidelines labels Aug 4, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 261c458af6e3eed5d099144aff95d2b5035f656b -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/w-/workflow.json b/versions/w-/workflow.json
index 725cdd2..c621401 100644
--- a/versions/w-/workflow.json
+++ b/versions/w-/workflow.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "8a8abc2f5725ad78500a5edd20df2c73ee2ddc37",
+      "git-tree": "ab1da719c3684f213ddd140aa6e09994c73580ee",
       "version": "0.9.6",
       "port-version": 0
     },

@dengjun101
Copy link
Contributor Author

I have communicated with the maintainer of workflow. We think this REF is consistent with the Release v0.9.6 on branch master.
We made a tag v0.9.6-win for windows branch. PTAL

@dengjun101 dengjun101 requested a review from JonLiu1993 August 4, 2021 06:05
@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Aug 4, 2021
@dan-shaw dan-shaw merged commit 3fbca9e into microsoft:master Aug 5, 2021
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.

4 participants