Skip to content

Contributing

Ryan Johnson edited this page Sep 10, 2019 · 3 revisions

draft

Submitting a PR

Keep all of the LGTM's in the PR comment

As you gain more experience, you'll be able to determine which are absolutely necessary, but for the first few PRs, keep all three (e.g., Dev LGTM, Design LGTM, Zoom LGTM).

Get familiar with squashing/rebasing commits via git.

More often than not, we will require a PR to be squashed to a single commit before merging.

squash: combine multiple commits into a single commit

Clone this wiki locally