Merge pull request #726 from sdboyer/maintainers

Maintainer's guide, et. all

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,23 @@
+<!--
+
+Thanks for filing an issue! If this is a question or feature request, just delete
+everthing here and write out the request, providing as much context as you can.
+
+-->
+
+### What version of Go (`go version`) and `dep` (`git describe --tags`) are you using?
+
+### What `dep` command did you run?
+
+<!--
+
+Paste the output of the commands you ran in here, making sure to pass -v for maximum context.
+
+The output of `dep hash-inputs` may also be helpful to include.
+
+-->
+
+### What did you expect to see?
+
+### What did you see instead?
+

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,19 @@
+<--
+Work-in-progress PRs are welcome as a way to get early feedback - just prefix
+the title with [WIP].
+-->
+
+### What does this do / why do we need it?
+
+### What should your reviewer look out for in this PR?
+
+### Do you need help or clarification on anything?
+
+### Which issue(s) does this PR fix?
+
+<--
+
+fixes #
+fixes #
+
+-->

CONTRIBUTING.md

@@ -1,6 +1,6 @@
-# Contributing to Dep
+# Contributing to `dep`
 
-Dep is an open source project.
+`dep` is an open source project.
 
 It is the work of hundreds of contributors. We appreciate your help!
 
@@ -12,9 +12,8 @@ Please check the existing issues and [FAQ](FAQ.md) to see if your feedback has a
 
 When [filing an issue](https://github.com/golang/dep/issues/new), make sure to answer these five questions:
 
-1. What version of Go are you using (`go version`)?
-2. What operating system and processor architecture are you using?
-3. What did you do?
+1. What version of Go (`go version`) and `dep` (`git describe --tags`) are you using??
+3. What `dep` command did you run?
 4. What did you expect to see?
 5. What did you see instead?
 
@@ -32,7 +31,7 @@ label highlights issues that are well-suited for folks to jump in on. The
 [good-first-pr](https://github.com/golang/dep/issues?q=is%3Aissue+is%3Aopen+label%3Agood-first-pr)
 label further identifies issues that are particularly well-sized for newcomers.
 
-Unless otherwise noted, the Dep source files are distributed under
+Unless otherwise noted, the `dep` source files are distributed under
 the BSD-style license found in the LICENSE file.
 
 All submissions, including submissions by project members, require review. We
@@ -52,3 +51,49 @@ your current agreements on file or to sign a new one.
 You generally only need to submit a CLA once, so if you've already submitted one
 (even if it was for a different project), you probably don't need to do it
 again.
+
+## Maintainer's Guide
+
+`dep` has subsystem maintainers; this guide is intended for them in performing their work as a maintainer.
+
+### General guidelines
+
+* _Be kind, respectful, and inclusive_. Really live [that CoC](https://github.com/golang/dep/blob/master/CODE_OF_CONDUCT.md). We've developed a reputation as one of the most welcoming and supportive project environments in the Go community, and we want to keep that up!
+* The lines of responsibility between maintainership areas can be fuzzy. Get to know your fellow maintainers - it's important to work _with_ them when an issue falls in this grey area.
+* Remember, the long-term goal of `dep` is to disappear into the `go` toolchain. That's going to be a challenging process, no matter what. Minimizing that eventual difficulty should be a guiding light for all your decisions today.
+  * Try to match the toolchain's assumptions as closely as possible ([example](https://github.com/golang/dep/issues/564#issuecomment-300994599)), and avoid introducing new rules the toolchain would later have to incorporate.
+  * Every new flag or option in the metadata files is more exposed surface area that demands conversion later. Only add these with a clear design plan.
+  * `dep` is experimental, but increasingly only on a larger scale. Experiments need clear hypotheses and parameters for testing - nothing off-the-cuff.
+* Being a maintainer doesn't mean you're always right. Admitting when you've made a mistake keeps the code flowing, the environment health, and the respect level up.
+* It's fine if you need to step back from maintainership responsibilities - just, please, don't fade away! Let other maintainers know what's going on.
+
+### Issue management
+
+* We use [Zenhub](https://www.zenhub.com) to manage the queue, in addition to what we do with labels.
+  * Pipelines, and [the board](https://github.com/golang/dep#boards) are one thing we try to utilize:
+    * **New Issues Pipeline**: When someone creates a new issue, it goes here first. Keep an eye out for issues that fall into your area. Add labels to them, and if it's something we should do, put it in the `Backlog` pipeline. If you aren't sure, throw it in the `Icebox`. It helps to sort this pipeline by date.
+    * **Icebox Pipeline**: Issues that we aren't immediately closing but aren't really ready to be prioritized and started on. It's not a wontfix bucket, but a "not sure if we should/can fix right now" bucket.
+    * **Backlog Pipeline**: Issues that we know we want to tackle. You can drag/drop up and down to prioritize issues.
+  * Marking dependencies/blockers is also quite useful where appropriate; please do that.
+  * We use epics and milestones in roughly the same way (because OSS projects don't have real sprints). Epics should be duplicated as milestones; if there's a main epic issue, it should contain a checklist of the relevant issues to complete it.
+* The `area:` labels correspond to maintainership areas. Apply yours to any issues or PRs that fall under your purview. It's to be expected that multiple `area:` labels may be applied to a single issue.
+* The [`help-wanted`](https://github.com/golang/dep/issues?q=is%3Aissue+is%3Aopen+label%3Ahelp-wanted) and [`good-first-pr`](https://github.com/golang/dep/issues?q=is%3Aissue+is%3Aopen+label%3Agood-first-pr) labels are two of our most important tools for making the project accessible to newcomers - a key goal for our community. Here's how to use them well.
+  * `good-first-pr` should be applied when there's a very straightforward, self-contained task that is very unlikely to have any hidden complexity. The real purpose of these is to provide a "chink in the armor", providing newcomers a lens through which to start understanding the project.
+  * `help-wanted` should be applied to issues where there's a clear, stated goal, there is at most one significant question that needs answering, and it looks like the implementation won't be inordinately difficult, or disruptive to other parts of the system.
+    * `help-wanted` should also be applied to all `good-first-pr` issues - it's duplicative, but not doing so seems unfriendly.
+
+
+### Pull Requests
+
+* Try to make, and encourage, smaller pull requests.
+* [No is temporary. Yes is forever.](https://blog.jessfraz.com/post/the-art-of-closing/)
+* Long-running feature branches should generally be avoided. Discuss it with other maintainers first.
+* Unless it's trivial, don't merge your own PRs - ask another maintainer.
+* Commit messages should follow [Tim Pope's rules](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).
+* Checklist for merging PRs:
+  * Does the PR pass [the code review comments](https://github.com/golang/go/wiki/CodeReviewComments)? (internalize these rules!)
+  * Are there tests to cover new or changed behavior? Prefer reliable tests > no tests > flaky tests.
+  * Does the first post in the PR contain "Fixes #..." text for any issues it resolves?
+  * Are any necessary follow-up issues _already_ posted, prior to merging?
+  * Does this change entail the updating of any docs?
+     * For docs kept in the repo, e.g. FAQ.md, docs changes _must_ be submitted as part of the same PR.