[MacPorts] Meetings/MacPortsMeeting2018/GitHub-LessonsLearned added

[MacPorts]

Page "Meetings/MacPortsMeeting2018/GitHub-LessonsLearned" was added by neverpanic
Comment: += Lessons learned from GitHub
Content:
-------8<------8<------8<------8<------8<------8<------8<------8<--------
= Lessons Learned from the move to GitHub

== Repository Conversion
* Overall that went well
* The `contrib` repositories should have been split into smaller chunks. We should probably split them and archive the old repository.
* There are some user repositories left that are dead. We should set a deadline, send a mail to the list and remove all of them after the deadline.

== Pull Requests
* The bot is very helpful, and so is the GitHub UI for merging things easily without having to use a command line.
* pmetzger is doing a great job in keeping the number of open pull requests down
* Only members can be assigned to pull requests in GitHub, but we have non-member maintainers. We should change the bot to assign the PRs if the maintainers are members, but keep the current behavior to also support the use case for external maintainers.
* We may have a bit of a hot trigger finger when merging pull requests, but that does not seem to have caused problems so far, so it's likely a non-issue.
* We can now request reviews before merging changes and are using this quite a bit on base and infrastructure repositories.

== GitHub handles in Portfiles
* A lot of maintainers have not added their GitHub handle to Portfiles, even regular contributors.
* We could use the Trac database to compute a match and automate the task.
* We should probably make Trac always use email addresses from GitHub and remove the ability to change email addresses in Trac.

== Trac
* trac-github could monitor pull requests mentioning tickets and post a comment with a link to the pull requests.
* Trac's GitHub integration is nice so far, the unified login works well.
* The trac landing pages are not very good; some user experience improvements could be done. We'll defer discussion to the UX session that's planned for tomorrow.
* Should Trac send a periodic digest for new open tickets to macports-dev? We could use https://trac-hacks.org/ticket/8802 for that.

-------8<------8<------8<------8<------8<------8<------8<------8<--------

--
Page URL: <https://trac.macports.org/wiki/Meetings/MacPortsMeeting2018/GitHub-LessonsLearned>
MacPorts <https://www.macports.org/>
Ports system for macOS

This is an automated message. Someone added your email address to be
notified of changes on 'Meetings/MacPortsMeeting2018/GitHub-LessonsLearned' page.
If it was not you, please report to admin at macports.org.