-
Notifications
You must be signed in to change notification settings - Fork 25
Contributing
Ryan Johnson edited this page Sep 10, 2019
·
3 revisions
draft
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).
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