How to connect: https://notredame.zoom.us/j/94030214208 (link will launch Zoom client – if you do not have Zoom, expand the instructions below)
Expand | ||
---|---|---|
| ||
Meeting ID: 940 3021 4208 One tap mobile
|
Time: 9:00am PDT / Noon EDT
...
Roll call by timezone per following order - ensure notetaker is present (moderator)
folks outside North and South America
Eastern timezone
Central timezone
Mountain timezone
Pacific timezone
folks who were missed or who dialed in during roll call
- Remind everyone to sign in on agenda.
- Welcome all newcomers!
- Agenda (moderator)
- Call for new agenda items (moderator)
- Deprecation message style guide
- Should we make one?
- Passive third-person vs. accountable "we"
- Normalizing location of CONTRIBUTING document (see related PR for discussion) (Jeremy Friesen )
- Samvera help follow-up
- Custom renderer is ignored -- what do I need to look for? https://groups.google.com/g/samvera-tech/c/JZ2q2ZNe73s/m/qc8qiXrqBwAJ?pli=1
- Custom renderer is ignored -- what do I need to look for? https://groups.google.com/g/samvera-tech/c/JZ2q2ZNe73s/m/qc8qiXrqBwAJ?pli=1
- Pull request review
- https://github.com/samvera/hyrax/pull/4158 (Jeremy Friesen) I want to look at the failing specs
- https://github.com/samvera/hyrax/pull/4397 (Jeremy Friesen) I'd like to see if others get these errors locally
- https://github.com/samvera/hyrax/pull/4402 (Lynette Rayle) Move valkyrie create strategy to share specs
- https://github.com/samvera/hyrax/pull/4403 (Lynette Rayle) Fix comments and one test that was testing the wrong thing
- Moderator & notetaker for next time
- Moderator:
- Notetaker:
- After call, this week's notetaker should create the agenda for the next call:
Open template agenda titled "Samvera Tech Call 2020-xx-xx"
- Click on ... in the top right corner, and select copy.
- Popup will open for location. It should contain:
- Space: Samvera
- Parent page: 2020
- Select copy. New page should be created.
- Modify the title to remove "copy of", update it with the next date, add moderator, notetaker, and any carry-over agenda info. Click Publish.
- PR Review
- Review issues:
- PR review coordinator for next time:
...
- Deprecation Message Style
- Jeremy submitted a PR where a method was deprecated, active voice used, "we will be deprecating"
- Question: Who is "we"?
- Passive voice: "Will be deprecated" is used elsewhere, there is a lack lack of accountability
- Active voice has more accountability, but perhaps shouldn't be "we"
- Perhaps "Samvera will deprecate..."
- Doesn't a deprecation message already give some indication as to where in the code base the message is originating?
- Yes, usually the line is referenced
- Some feel as if determining whether "we" is used might not that be that important, the deprecation warning itself is the essential factor
- Also, ensuring that the line of code where the deprecation message originates should be in place
- Conclusion: Usage of "We" is inconclusive
- Renaming GitHub repository `master` branches
- Connotations of slavery with the usage of `master`
- Emory is switching to using `main`, Cornell supports this change
- We should be mindful of links to URLs containing branch names in documentation or elsewhere
- There should be a time period for when this is occurring
- We can use this opportunity to then also explain to the community why this change is being introduced
- Emory: Created a separate "main" branch without deleting "master", and fast-forward pushing commits to "main"
- git merge --ff-only
- In the process of issuing pull requests against "main"
- Still in the process of switching to "main" as the default
- Will retain "master" for a time period, just to ensure backward compatibility
- Samvera should frame the rationale, and then draft a series of action items for the migration plan
- Jeremy volunteered to write the framing document