Skip to content

Instantly share code, notes, and snippets.

@hardyoyo
Last active December 25, 2015 07:19
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save hardyoyo/898e9275d8e7a7aacca8 to your computer and use it in GitHub Desktop.
Save hardyoyo/898e9275d8e7a7aacca8 to your computer and use it in GitHub Desktop.
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