-
-
Save hardyoyo/898e9275d8e7a7aacca8 to your computer and use it in GitHub Desktop.
dspace 4.0 PR review results
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Manually culled from meeting transcript for 10/09/2013 (http://irclogs.duraspace.org/index.php?date=2013-10-09) | |
047: NO | un-mergeable | |
050: NO | un-mergeable | |
051: NO | un-mergeable | |
061: NO | un-mergeable | |
078: NO | un-mergeable | |
094: NO | obsoleted by PR #247 | |
159: NO | rejected | |
183: bugfix | |
222/186: related, mhwood and bollini are investigating | |
190: NO | unmergeable | |
192: bugfix | |
195: bugfix | |
196: bugfix, but discussed at end of 10/11/2013 meeting, pre-approved to merge if passes testing by hpottinger | |
202: NO | needs discussion | |
Automatically pulled from meeting transcript for 10/11/2013 (http://irclogs.duraspace.org/index.php?date=2013-10-11) | |
205: YES | pre-approved to merge (we have +3) | |
222: YES | pre-approved to merge | |
232: YES | pre-approved to merge | |
244: NO | Need further work | |
260: NO | need rebase | |
266: NO | Need rebase | |
287: YES | pre-approved after solving merge conflict / issue | |
294: YES | pre-approved merge after check | |
303: YES | at the end of meeting | |
304: NO | build error | |
308: YES | take caution about httpcomponent version | |
308: YES | pre-approved to merge (caution about http component version) | |
309: YES | merge at the end of meeting (I'm taking note) | |
309: YES | after check with creator | |
312: YES | see 308 | |
314: YES | pre-approved merge | |
316: NO | Need discussion | |
317: YES | pre-approved to merge | |
319: YES | pre-approved merge | |
320: YES | pre-approved merge | |
321: YES | pre-approved to merge with notice to mailing list | |
323: YES | pre-approved to merge | |
324: YES | pre-approved with jspui porting | |
196: YES | pre-approved, pending results of testing by hpottinger |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment