Re: Review Request 20407: AURORA-323:Add ability to merge in github pull requests similar to rbt
> On April 25, 2014, 4:06 a.m., Bill Farner wrote: > > I have some reservations with this, but they're not well-formed. I like > > the idea of giving committers a chance to chime in on reviews, and we seem > > to be building process around reviewboard for that. We also have plans to > > build test automation around reviewboard. This seems to turn github into a > > "back door" of sorts for changes to not receive the same level of scrutiny. > > What are your thoughts? > > Jake Farrell wrote: > Github makes it easier for some to contribute patches, but we should > require the same criteria as we do with reviewboard (Jira ticket, tests > performed, etc). This tooling is just to make it easier for us to manage > those contributions, wish we could disable Github pull requests on the > mirrors, but that is not an available option > > Bill Farner wrote: > I've obviously been delinquent here. My apologies. > > Overall -1 on this feature. > > I'm not fond of the idea of pulling diffs from different places. The > biggest issue for me is that we lose a canonical location for code reviews. > A big downside is that we would no longer have a cohesive history of the code > (i consider code reviews a non-trivial part of the history of project). It > also complicates future automation like code review validation on commits, > which i am very interested in doing to keep our commit history clean and > enforce agreed-upon best practices. I'm fine not including this as long as we have a documented process and stick to it - Jake --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20407/#review41435 --- On April 17, 2014, 4 a.m., Jake Farrell wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20407/ > --- > > (Updated April 17, 2014, 4 a.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-323 > https://issues.apache.org/jira/browse/AURORA-323 > > > Repository: aurora > > > Description > --- > > AURORA-323:Add ability to merge in github pull requests similar to rbt > > Created a new extension for rbt which adds a new github subcommand to preview > and > merge/commit available pull requests. Adds text to the commit message > so when github mirrors the given commit the pull request will automatically > be closed. > > > Diffs > - > > .reviewboardrc a66e6fd0da9102f760381401b56b41b29c8d6f4d > rbt b03c87f9df129a03cced6ec088aa5ed3e16edfe0 > > Diff: https://reviews.apache.org/r/20407/diff/ > > > Testing > --- > > Preview a pull request patch > ./rbt github -p 7 > > Commit a pull request > ./rbt github -c 7 > > > Thanks, > > Jake Farrell > >
Re: Review Request 20407: AURORA-323:Add ability to merge in github pull requests similar to rbt
> On April 25, 2014, 4:06 a.m., Bill Farner wrote: > > I have some reservations with this, but they're not well-formed. I like > > the idea of giving committers a chance to chime in on reviews, and we seem > > to be building process around reviewboard for that. We also have plans to > > build test automation around reviewboard. This seems to turn github into a > > "back door" of sorts for changes to not receive the same level of scrutiny. > > What are your thoughts? > > Jake Farrell wrote: > Github makes it easier for some to contribute patches, but we should > require the same criteria as we do with reviewboard (Jira ticket, tests > performed, etc). This tooling is just to make it easier for us to manage > those contributions, wish we could disable Github pull requests on the > mirrors, but that is not an available option I've obviously been delinquent here. My apologies. Overall -1 on this feature. I'm not fond of the idea of pulling diffs from different places. The biggest issue for me is that we lose a canonical location for code reviews. A big downside is that we would no longer have a cohesive history of the code (i consider code reviews a non-trivial part of the history of project). It also complicates future automation like code review validation on commits, which i am very interested in doing to keep our commit history clean and enforce agreed-upon best practices. - Bill --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20407/#review41435 --- On April 17, 2014, 4 a.m., Jake Farrell wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20407/ > --- > > (Updated April 17, 2014, 4 a.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-323 > https://issues.apache.org/jira/browse/AURORA-323 > > > Repository: aurora > > > Description > --- > > AURORA-323:Add ability to merge in github pull requests similar to rbt > > Created a new extension for rbt which adds a new github subcommand to preview > and > merge/commit available pull requests. Adds text to the commit message > so when github mirrors the given commit the pull request will automatically > be closed. > > > Diffs > - > > .reviewboardrc a66e6fd0da9102f760381401b56b41b29c8d6f4d > rbt b03c87f9df129a03cced6ec088aa5ed3e16edfe0 > > Diff: https://reviews.apache.org/r/20407/diff/ > > > Testing > --- > > Preview a pull request patch > ./rbt github -p 7 > > Commit a pull request > ./rbt github -c 7 > > > Thanks, > > Jake Farrell > >
Re: Review Request 20407: AURORA-323:Add ability to merge in github pull requests similar to rbt
> On April 25, 2014, 4:06 a.m., Bill Farner wrote: > > I have some reservations with this, but they're not well-formed. I like > > the idea of giving committers a chance to chime in on reviews, and we seem > > to be building process around reviewboard for that. We also have plans to > > build test automation around reviewboard. This seems to turn github into a > > "back door" of sorts for changes to not receive the same level of scrutiny. > > What are your thoughts? Github makes it easier for some to contribute patches, but we should require the same criteria as we do with reviewboard (Jira ticket, tests performed, etc). This tooling is just to make it easier for us to manage those contributions, wish we could disable Github pull requests on the mirrors, but that is not an available option - Jake --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20407/#review41435 --- On April 17, 2014, 4 a.m., Jake Farrell wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20407/ > --- > > (Updated April 17, 2014, 4 a.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-323 > https://issues.apache.org/jira/browse/AURORA-323 > > > Repository: aurora > > > Description > --- > > AURORA-323:Add ability to merge in github pull requests similar to rbt > > Created a new extension for rbt which adds a new github subcommand to preview > and > merge/commit available pull requests. Adds text to the commit message > so when github mirrors the given commit the pull request will automatically > be closed. > > > Diffs > - > > .reviewboardrc a66e6fd0da9102f760381401b56b41b29c8d6f4d > rbt b03c87f9df129a03cced6ec088aa5ed3e16edfe0 > > Diff: https://reviews.apache.org/r/20407/diff/ > > > Testing > --- > > Preview a pull request patch > ./rbt github -p 7 > > Commit a pull request > ./rbt github -c 7 > > > Thanks, > > Jake Farrell > >
Re: Review Request 20407: AURORA-323:Add ability to merge in github pull requests similar to rbt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20407/#review41435 --- I have some reservations with this, but they're not well-formed. I like the idea of giving committers a chance to chime in on reviews, and we seem to be building process around reviewboard for that. We also have plans to build test automation around reviewboard. This seems to turn github into a "back door" of sorts for changes to not receive the same level of scrutiny. What are your thoughts? - Bill Farner On April 17, 2014, 4 a.m., Jake Farrell wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20407/ > --- > > (Updated April 17, 2014, 4 a.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-323 > https://issues.apache.org/jira/browse/AURORA-323 > > > Repository: aurora > > > Description > --- > > AURORA-323:Add ability to merge in github pull requests similar to rbt > > Created a new extension for rbt which adds a new github subcommand to preview > and > merge/commit available pull requests. Adds text to the commit message > so when github mirrors the given commit the pull request will automatically > be closed. > > > Diffs > - > > .reviewboardrc a66e6fd0da9102f760381401b56b41b29c8d6f4d > rbt b03c87f9df129a03cced6ec088aa5ed3e16edfe0 > > Diff: https://reviews.apache.org/r/20407/diff/ > > > Testing > --- > > Preview a pull request patch > ./rbt github -p 7 > > Commit a pull request > ./rbt github -c 7 > > > Thanks, > > Jake Farrell > >
Re: Review Request 20407: AURORA-323:Add ability to merge in github pull requests similar to rbt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20407/ --- (Updated April 17, 2014, 4 a.m.) Review request for Aurora, Kevin Sweeney and Bill Farner. Changes --- instead of including this as an Aurora specific rbt extension i've made it generic and published it on pypi Bugs: AURORA-323 https://issues.apache.org/jira/browse/AURORA-323 Repository: aurora Description --- AURORA-323:Add ability to merge in github pull requests similar to rbt Created a new extension for rbt which adds a new github subcommand to preview and merge/commit available pull requests. Adds text to the commit message so when github mirrors the given commit the pull request will automatically be closed. Diffs (updated) - .reviewboardrc a66e6fd0da9102f760381401b56b41b29c8d6f4d rbt b03c87f9df129a03cced6ec088aa5ed3e16edfe0 Diff: https://reviews.apache.org/r/20407/diff/ Testing --- Preview a pull request patch ./rbt github -p 7 Commit a pull request ./rbt github -c 7 Thanks, Jake Farrell