-
Notifications
You must be signed in to change notification settings - Fork 26
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