Skip to content

Instantly share code, notes, and snippets.

@hardyoyo hardyoyo/gist:898e9275d8e7a7aacca8 Secret
Last active Dec 25, 2015

Embed
What would you like to do?
dspace 4.0 PR review results
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
You can’t perform that action at this time.