[GitHub] metron pull request #902: METRON-1413 Add Metron Commit Tool

2018-01-20 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/902 ---

[GitHub] metron issue #902: METRON-1413 Add Metron Commit Tool

2018-01-20 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/902 Also, I've never liked the name `prepare-commit`. If you guys can think of something better, please let me know. ---

[GitHub] metron pull request #902: METRON-1413 Add Metron Commit Tool

2018-01-20 Thread nickwallen
GitHub user nickwallen reopened a pull request: https://github.com/apache/metron/pull/902 METRON-1413 Add Metron Commit Tool This PR contributes the `prepare-merge` tool that many (some?) contributors use. Up until now, it has been managed in a separate repo. I didn't

[GitHub] metron issue #902: METRON-1413 Add Metron Commit Tool

2018-01-20 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/902 Travis had a hickup... Doing the "close/reopen" dance to trigger the CI build. ---

[GitHub] metron issue #902: METRON-1413 Add Metron Commit Tool

2018-01-20 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/902 The Apache ID isn't really necessary to use `prepare-commit`. What if I just changed the docs to note that it is not required? I could document how you could use the script to review a PR.

[GitHub] metron issue #902: METRON-1413 Add Metron Commit Tool

2018-01-20 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/902 > @JonZeolla: ... the curl command HTML encodes the JIRA description but I don't know of a clean bash-native way to decode it. Yes, I have noticed, especially with apostrophes. When that