Skip to content

Latest commit

 

History

History
108 lines (77 loc) · 4.4 KB

CONTRIBUTING.md

File metadata and controls

108 lines (77 loc) · 4.4 KB

Contributing to @percy/cli

✨ Thanks for contributing to @percy/cli! ✨

As a contributor, here are the guidelines we would like you to follow:

We also recommend that you read How to Contribute to Open Source.

Submitting a Pull Request

Good pull requests, whether patches, improvements, or new features, are a fantastic help. They should remain focused in scope and avoid containing unrelated commits.

Please ask first before embarking on any significant pull requests (e.g. implementing features, refactoring code), otherwise you risk spending a lot of time working on something that the project's developers might not want to merge into the project.

If you have never created a pull request before, welcome 🎉 😄. Here is a great tutorial on how to send one :)

Here is a summary of the steps to follow:

  1. Set up the workspace
  2. If you cloned a while ago, get the latest changes from upstream and update dependencies:
$ git checkout master
$ git pull upstream master
$ rm -rf node_modules
$ yarn
  1. Create a new topic branch (off the main project development branch) to contain your feature, change, or fix:
$ git checkout -b <topic-branch-name>
  1. Make your code changes, following the Coding rules
  2. Push your topic branch up to your fork:
$ git push origin <topic-branch-name>
  1. Open a Pull Request with a clear title and description.

Tips:

Coding rules

Source code

To ensure consistency and quality throughout the source code, all code modifications must have:

  • No linting errors
  • A test for every possible case introduced by your code change
  • We in general expect e2e test cases with minimum mocking, mocking is discouraged unless there is no other way of testing the same.
  • 100% test coverage
  • Valid commit message(s)

Commit message guidelines

Atomic commits

If possible, make atomic commits, which means:

  • a commit should contain exactly one self-contained functional change
  • a functional change should be contained in exactly one commit
  • a commit should not create an inconsistent state (such as test errors, linting errors, partial fix, feature without documentation etc...)

A complex feature can be broken down into multiple commits as long as each one maintains a consistent state and consists of a self-contained change. We further always squash and merge commits to make sure that on base branch the commit history looks good and is easy to understand all changes for a specific feature.

Working with the code

Set up the workspace

Fork the project, clone your fork, configure the remotes and install the dependencies:

# Clone your fork of the repo into the current directory
$ git clone https://github.com/percy/cli
# Navigate to the newly cloned directory
$ cd cli
# Assign the original repo to a remote called "upstream"
$ git remote add upstream https://github.com/percy/cli
# Install the dependencies
$ yarn

Lint

@percy/cli uses eslint for linting.

Before pushing your code changes make sure there are no linting errors with yarn lint.

Tests

You can run the tests with:

$ yarn test

You can limit tests for a specific package with:

$ yarn workspace @percy/core test

And get the coverage number for tests with:

$ yarn test:coverage