# Contributing

VertoTrade is an open-source project. If you want to contribute to the project, this section is here to guide you through your first steps with the VertoTrade team.

Before starting any development, we highly encourage you to submit an issue on Github in order to discuss the problem, and the solution with the team.

# Setup your dev environment

Install yarn If you haven't.

  1. Fork and clone the repository

    $ git clone [fork_repo_url]
  2. Add upstream remote. E.g.

    $ git remote add upstream git@github.com:vertotrade/verto.ui.git
  3. Make sure you have the latest version of the default branch ( develop )

    $ git checkout develop
    $ git pull upstream develop
  4. Create your own branch and install dependencies

    $ git checkout -b branch-name
    $ yarn
  5. Happy coding 🎉

# Coding rules

We try to maintain as much consistency as we can between each of our repository. Your pull request has more chances to be accepted if you follow the following rules, and write high quality code. Let's get started 💪

# Use the UIKit

If a variant of a component needs to be created, use the corresponding component in the UI Kit as a base. For example:

import styled from 'styled-components'
import { Button } from '@vertotrade/uikit'

const NewButtonVariant = styled(Button)`
  // custom styles here

# Use the tools!

Most of our repos use Typescript, ESLint, and Prettier. Make sure you're familiar with Typescript’s best practices and enable an ESLint and Prettier plugin for your IDE.

# Some good practices

  • Keep components as small and "dumb" as possible.
  • Use Composition over Inheritance.
  • Keep in mind that your code will be read and maintained by several other developers. Make it as clear and easy to update as possible.

# Creating your pull request

Your code is ready to be submitted for review, congratulations!

  • All pull requests must have a description of what the PR is trying to accomplish.
  • Keep pull requests as small as possible. Larger pull requests should be broken up into smaller chunks with a dedicated base epic branch. Please tag the PR's that are merging into your base branch with the epic tag.
  • If possible self-review your PR and add comments where additional clarification is needed.

# Pull Request Title

Our Pull Request Title follow Conventional Commits using commitlint.‌

More at Angular's guidelines__

Type Description
build Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
ci Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
docs Documentation only changes
feat A new feature
fix A bug fix
perf A code change that improves performance
refactor A code change that neither fixes a bug nor adds a feature
style Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
test Adding missing tests or correcting existing tests

Thanks for helping us making VertoTrade even more awesome