So Rodrigo just broke the gitlab ci on libdrm, I figured perfect time to start this discussion.
There's imo 2 reasons to do this: - No more "oops, I broke make check". If we use gitlab merge requests gitlab CI will test everything, and we can set 2 checkboxes that disallow direct pushes (i.e. require merge requests), and that all merge requests must pass CI first. - maintainer-tools is a small project with a small team and little activity. Much easier to figure out the details here than in one of our big projects. And there's lots to figure out, which we need to be able to explain (and have documented) for our 50+ team, if we ever want to use gitlab: commandline tools, emacs modes, best practices for setup, ... To avoid this being an entirely hypothetical discussion, I've gone ahead and created a merge request for this: https://gitlab.freedesktop.org/drm/maintainer-tools/merge_requests/3 For keeping up with activity: Go to the main repo, click the alarm icon (which probably says "Global" now), and change the setting to "Watch". That should keep you updated on all issues and merge requests, like with a mailing list. v2: Add link to merge request. Cc: Jani Nikula <[email protected]> Cc: Sean Paul <[email protected]> Cc: Rodrigo Vivi <[email protected]> Cc: Lucas De Marchi <[email protected]> Signed-off-by: Daniel Vetter <[email protected]> --- CONTRIBUTING.rst | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index c7846e318b7e..636d94e3c0af 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -1,11 +1,21 @@ CONTRIBUTING ============ -Submit patches, bug reports, and questions for any of the maintainer tools and -documentation to the [email protected] mailing list. +Patches should be sent via `GitLab merge requests +<https://docs.gitlab.com/ce/gitlab-basics/add-merge-request.html>`_. +maintainer-tools are hosted on `freedesktop.org's GitLab +<https://gitlab.freedesktop.org/drm/maintainer-tools>`_: in order to submit +code, you should create an account on this GitLab instance, fork the core Weston +repository, push your changes to a branch in your new repository, and then +submit these patches for review through a merge request. -Please make sure your patches pass the build and self tests by running:: +Gitlab CI will automatically run all tests. You can test your patches locally by +running:: $ make check -Push the patches once you have an ack from maintainers (Jani/Daniel). +All merge requests need an ack from at least one of the committers before it can +be pushed. Don't push to master directly. + +Bug reports, suggestions for improvements and questions for any of the +maintainer tools and documentation should be filed as new issues. -- 2.19.0.rc2 _______________________________________________ dim-tools mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dim-tools
