[PATCH] D22463: [RFC] Moving to GitHub Proposal: NOT DECISION!

Dean Michael Berris via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 18 20:17:43 PDT 2016


dberris added a subscriber: dberris.
dberris added a comment.

Mostly wording comments, thank you for writing this up!


================
Comment at: docs/Proposals/GitHub.rst:78
@@ +77,3 @@
+
+GitHub, like GitLab and BitBucket, provide FREE code hosting for open source
+projects. Essentially, they will completely replace *all* the infrastructure that
----------------
nit: I see you use FREE in caps but this instance isn't *FREE* (as opposed to the first mention above) -- consider making it consistent? Either remove the emphasis (just "free") or emphasise consistently?

================
Comment at: docs/Proposals/GitHub.rst:86
@@ +85,3 @@
+for example development meetings, sponsoring disadvantaged people to work on
+compilers and foster diversity and quality in our community.
+
----------------
Did you mean "diversity and equality" instead of "diversity and quality" here?

================
Comment at: docs/Proposals/GitHub.rst:110-112
@@ +109,5 @@
+
+As with the current SVN and Git repositories, each project will be separate,
+on its own, and people will also be able to check them out in the same way
+they're already doing today.
+
----------------
Consider rewording this sentence -- it's a little too long and is trying to say too many things.

Perhaps something like:

"Each LLVM project will continue to be hosted as separate GitHub repositories under a single GitHub organisation. Users can continue to choose to use either SVN or Git to access the repositories to suit their current workflow."

================
Comment at: docs/Proposals/GitHub.rst:185
@@ +184,3 @@
+
+Are there any other upstream systems that could be affected?
+
----------------
Probably worth mentioning how Phabricator will need to be updated to integrate with the GitHub repository once the canonical repo is changed.


https://reviews.llvm.org/D22463





More information about the cfe-commits mailing list