-
Notifications
You must be signed in to change notification settings - Fork 26
Contributing
Ryan Johnson edited this page Sep 10, 2019
·
3 revisions
draft
Keep all of the LGTM's in the PR comment
As you begin to make your first PR contributions, leave all LGTMs in the PR description (e.g., Dev LGTM, Design LGTM, Zoom LGTM). When you gain more experience, you'll be able to determine which LGTMs are absolutely necessary.
Get familiar with how to squash multiple commits into a single commit using git. More often than not, a PR will need to be squashed down to a single commit before merging.