[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-04-13 Thread anyway1021 
Github user anyway1021 closed the pull request at:

https://github.com/apache/incubator-eagle/pull/42


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-04-13 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-209364784
  
Please close the pull request because it has been merged.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-03-14 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-196246105
  
@anyway1021 thanks very much for the contribution, but it seems that the 
script is too complex for daily  development work. I can't succeed to process 
any PR until now.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-01-17 Thread anyway1021 
Github user anyway1021 closed the pull request at:

https://github.com/apache/incubator-eagle/pull/42


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-01-17 Thread anyway1021 
GitHub user anyway1021 reopened a pull request:

https://github.com/apache/incubator-eagle/pull/42

EAGLE-84 create first version of auto_merge.py under folder eagle-dev

This version doesn't include the logic of checking unit test results 
reported by CI jobs. The purposes of this pull request are:
1. create sub-directory of "eagle-dev" in the project's root directory to 
contain dev related tools or scripts.
2. contribute this first version so that people can use it instead of 
manually scripting jobs to merge, edit, commit and push.
3. let this script receive widely review.

The integration of Github & CI is under way, and I will update this script 
once I'm aware enough of the verification logic for unit test results, soon.

Any code review and suggestion will be appreciated.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/anyway1021/incubator-eagle 
auto_merge_python_script

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-eagle/pull/42.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #42


commit 4f36ef15243ffc30ff30cbc893f3388ea8f27685
Author: ziwu 
Date:   2015-12-28T15:32:24Z

create first version of auto_merge.py under folder eagle-dev

commit 5aa4e21978a321009ecd41fb03e9c9f99884f0e6
Author: ziwu 
Date:   2015-12-29T09:24:37Z

change code with python2.7 syntax

commit b835748d1e1399ee4b57d42364ce62b3be2b180f
Author: ziwu 
Date:   2015-12-29T09:27:16Z

rename script under eagle-dev folder: auto_merge.py --> 
merge_pull_request.py

commit 3bee34b9d035f3dc0e4d99028bec266acca68f78
Author: anyway1021 
Date:   2015-12-29T14:32:47Z

modify operting remote alias and temp folder name

commit ad31425fbf09d16ac2625953aba33ed50480502a
Author: anyway1021 
Date:   2016-01-03T11:08:02Z

add --log-level argument to control log level

commit 9477d6e072bd9c0e1342a9e6d3193f6c9ee3
Author: anyway1021 
Date:   2016-01-03T11:25:49Z

add debug step in run_command()

commit ee5268325ceefd750ce60a7dc5c473cb91e52202
Author: anyway1021 
Date:   2016-01-04T04:04:59Z

additionally debug execution result in run_command()

commit 8ea33ab0f640a9ad9a65fd81b3a319313cf0c23b
Author: anyway1021 
Date:   2016-01-04T06:58:33Z

lookup lastest_commit_timestamp from pr's commits, not from pr_info

commit 4bc48f515a6d60439d63d352bdc05cfa77f760bd
Author: anyway1021 
Date:   2016-01-04T07:36:02Z

add argument for patch mode and adjust the help message, adjust some method 
names, make pr_number a global variable, and do refactoring

commit c36095dff6c360a5a05615be36a7a200d13bf26e
Author: anyway1021 
Date:   2016-01-06T03:40:13Z

finished refactoring, add the non-patch mode

commit 556871134d5ab2a0111263b7dcd53a54d2399b3c
Author: anyway1021 
Date:   2016-01-08T08:33:18Z

fix os separator incompatibility between slash and back-slash

commit 359546f91f343094ee64ce26ba574183b5f82b57
Author: anyway1021 
Date:   2016-01-10T15:24:58Z

modify commit message input prompt for fit windows system and other system

commit 1a05758d8f82e7d54a006124898ef18564f06c7e
Author: anyway1021 
Date:   2016-01-12T04:03:27Z

modify pr_title parsing regexp, making it tolerant char-case and marks 
after jira-id

commit 58f754474003ca11083a25cb176f04d25f448d11
Author: anyway1021 
Date:   2016-01-12T06:47:54Z

change to get author email from commits of the PR, instead of directly 
querying author info




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-01-10 Thread anyway1021 
Github user anyway1021 commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-170446877
  
Updates made until commit "359546f":
1. Embed debug messages when the script is executed with argument “-l 
debug” or “--log-level debug”, other wise, it only output “info” log.
2. Enhance the error handling and env-recovery mechanism.
3. Add non-patch mode to accomplish the goal. If the script is executed 
with a “-p” or “--patch” argument, it will use the patch mode, 
otherwise it uses non-patch mode by default.
4. Improve the compatibility for running it in windows and mac system.
5. Enrich help message for better instructions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-01-10 Thread anyway1021 
Github user anyway1021 commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-170446226
  
@haoch 

The error of #51 occurs because the title of the PR is not in the expected 
format, it starts with "EAGLE-114:" but not "EAGLE-114", there is an additional 
":"
For sure, it's possible to support passing jira-id, do you mean in this way 
we don't have to check the title, but instead form a correct title by the 
script itself? I think this is a good idea, but do we need to check the 
authenticity of the jira id? Please confirm.

BTW, did you mean to get author's email from the info in patch file or some 
other place? Actually we can get a lot of info from a pr url, but emails may 
vary in different places. E.g. my global git config takes company's account and 
email as a user, but for apache projects, I have to config the local user with 
github accounts, if anyone forget this, the email info in commits will be 
different. So, which will be better reliable?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-01-10 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-170434434
  
Could we get the author email from commit patch instead of github account 
api?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2016-01-10 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-170425595
  
Failed to merge "https://github.com/apache/incubator-eagle/pull/51"; due to 
`ERR: the title of the pull request should start with "EAGLE-${jira_id}", 
followed by a white-space and textual content, please revise accordingly, exit 
with code -1 ...`, maybe it possible to support give jirap-id by parameters?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread anyway1021 
Github user anyway1021 commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-167723365
  
Sure, Hao, that's a good suggestion and the old name was not defined as the
final one, i'll change these things and add commits to the PR.

On Tue, Dec 29, 2015 at 11:57 AM, Hao Chen  wrote:

> And how do you think rename to script to a more meaningful name say
> merge_pull_request.py instead of auto_merge.py? Because the script is
> actually executed manually but not automatically.
>
> —
> Reply to this email directly or view it on GitHub
> 
> .
>



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-167714742
  
And how do you think rename to script to a more meaningful name say 
`merge_pull_request.py` instead of `auto_merge.py`? Because the script is 
actually executed manually but not automatically.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-167714573
  
Please use python-2.7 instead of python-3.*


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread haoch 
Github user haoch commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-167712072
  
LGTM, will test for some days before merging into code base. Thanks for the 
contribution. @anyway1021 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread anyway1021 
Github user anyway1021 commented on the pull request:

https://github.com/apache/incubator-eagle/pull/42#issuecomment-167688948
  
This is a copy of an experimentation execution log of the script, which is 
hoped to help reviewers (git-repo-url, ticket-id are mocked, not 
apache/incubator-eagle):
 START PROCESSING 
INF: query pr information from github...
INF: will use "am_eagle_remote" as a remote name to operate
INF: looking up comments from github...
INF: verified: this pr is reviewed by ['anyway1021'] and finally approved 
by ['anyway1021'], it's good to merge
INF: querying email address for reviewers' account from github...
INF: downloading 17.patch from github...
IPT: please input the description of the merge, which will be regarded as a 
part of the final commit message
( to start new lines,  to finish inputting):
recover 
INF: committed to local repository on branch 
operating_branch_1451350746196985, the commit history now looks like: 
new ->  * commit 643dc13d5560eae734ea1e9ac166d68adf363113 (HEAD, 
operating_branch_1451350746196985)
| Author: ziwu 
| Date:   Tue Dec 29 08:59:39 2015 +0800
| 
| EAGLE-31 recover toString() of Address.java
| recover
| 
| https://issues.apache.org/jira/browse/EAGLE-31
| 
| Author: @anyway1021 
| Reviewer(s): @anyway1021 
| 
| Closes #17.
|  
old ->  * commit 2a935badad668b97de7c8e8a9150786c88fabb8d 
(am_eagle_remote/master)
| Author: ziwu 
| Date:   Mon Dec 28 22:58:14 2015 +0800
| 
| EAGLE-31 add street1 info to toString() of Address.java
| ddaabb
| 
| https://issues.apache.org/jira/browse/EAGLE-31
| 
| Author: @anyway1021 
| Reviewer(s): @anyway1021 
| 
| Closes #16.
* commit ..
IPT: shall we push commit <643dc13d5560eae734ea1e9ac166d68adf363113> to 
remote ?
():y
INF: pushing to remote repository...
INF: pull request <17> has been applied and pushed to remote 
 of repository 

INF: if you're going to continue with any forked repository, please update 
it accordingly
INF: Bye.
 ENDED PROCESSING 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread anyway1021 
Github user anyway1021 closed the pull request at:

https://github.com/apache/incubator-eagle/pull/42


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread anyway1021 
GitHub user anyway1021 reopened a pull request:

https://github.com/apache/incubator-eagle/pull/42

EAGLE-84 create first version of auto_merge.py under folder eagle-dev

This version doesn't include the logic of checking unit test results 
reported by CI jobs. The purposes of this pull request are:
1. create sub-directory of "eagle-dev" in the project's root directory to 
contain dev related tools or scripts.
2. contribute this first version so that people can use it instead of 
manually scripting jobs to merge, edit, commit and push.
3. let this script receive widely review.

The integration of Github & CI is under way, and I will update this script 
once I'm aware enough of the verification logic for unit test results, soon.

Any code review and suggestion will be appreciated.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/anyway1021/incubator-eagle 
auto_merge_python_script

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-eagle/pull/42.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #42


commit 4f36ef15243ffc30ff30cbc893f3388ea8f27685
Author: ziwu 
Date:   2015-12-28T15:32:24Z

create first version of auto_merge.py under folder eagle-dev




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-84 create first version of aut...

2015-12-28 Thread anyway1021 
GitHub user anyway1021 opened a pull request:

https://github.com/apache/incubator-eagle/pull/42

EAGLE-84 create first version of auto_merge.py under folder eagle-dev



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/anyway1021/incubator-eagle 
auto_merge_python_script

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-eagle/pull/42.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #42


commit 4f36ef15243ffc30ff30cbc893f3388ea8f27685
Author: ziwu 
Date:   2015-12-28T15:32:24Z

create first version of auto_merge.py under folder eagle-dev




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---