Samvera Tech Call 2019-07-10 - CANCELLED (Hyrax Pull Request Review Only)

How to connect: https://psu.zoom.us/j/613720745 (link will launch Zoom client – if you do not have Zoom, expand the instructions below)

Time: 9:00am PDT / Noon EDT

Moderator: tamsin woo

Notetaker: James Griffin

Attendees:

• Chris Colvard (Deactivated) (Indiana University)
• Collin Brittle (Emory University)
• Julie Allinson (U. London)
• tamsin woo (UC Santa Barbara)
• James Griffin (Princeton University)

Agenda

1. Roll call by timezone per following order - ensure notetaker is present (moderator)

   1. folks outside North and South America

   2. Eastern timezone

   3. Central timezone

   4. Mountain timezone

   5. Pacific timezone

   6. folks who were missed or who dialed in during roll call

   7. Remind everyone to sign in on agenda.
   8. Welcome all newcomers!
2. Agenda (moderator)
   1. Call for new agenda items (moderator)
   2. Add agenda items here
3. Moderator & notetaker for next time
   
   1. Moderator:
   2. Notetaker:
4. After call, this week's notetaker should create the agenda for the next call:

   1. Open template agenda titled "Samvera Tech Call 2019-xx-xx"

   2. Click on ... in the top right corner, and select copy.
   3. Popup will open for location. It should contain: 
      1. Space: Samvera
      2. Parent page: 2019
   4. Select copy. New page should be created.
   5. Modify the title to remove "copy of", update it with the next date, add moderator, notetaker, and any carry-over agenda info. Click Publish.
5. PR Review
   
   1. Review issues:
   2. PR review coordinator for next time: 

Notes

Outstanding Wings PRs are reviewed

There were some PRs which were stale and closed

Should they be reviewed?

If somebody is asking for one of them to be reviewed, then we should; otherwise if no one is advocating for a review the perhaps we should not

#3479: LaRita authored a sizeable PR

It never received a review

Chris asked LaRita about this in a GitHub comment

Julie: Has one waiting for review, but has merge conflicts which shall need to be reviewed (#3760)

Wanted a more thorough review before continuing to work to resolve conflicts upstream

Rob Kaufman approved of it more than one month ago

Did nothing substantive, but did introduce changes which dismissed the approving review

Will need to resolve conflicts before approving of the PR?

Questions regarding riiif Gem support arose during the review

If users haven't installed riiif, there might be an issue; this might need to be tested

Inline the ImageMagickInfoExtractor into the Hyrax namespace in order to avoid a hard dependency for Riiif::ImageMagickInfoExtractor