CONTRIBUTING.md

Contributing

Welcome! This document explains how you can contribute to making code-interview even better.

Getting Started

Installation

git clone <this repo>
npm install -g commitizen
npm install

Directory Structure

Code is organised into modules which contain one-or-more components. This a great way to ensure maintainable code by encapsulation of behavior logic. A component is basically a self contained app usually in a single file or a folder with each concern as a file: style, template, specs, e2e, and component class. Here's how it looks:

code-interview/
 ├──config/                       * configuration files live here (e.g. eslint, verify, testUnit)
 │
 ├──lib/                          * source code files should be here
 │
 ├──dist/                         * production-build code should live here
 │
 ├──reports/                      * test reports appear here
 │
 ├──test/                         * unit test specifications live here
 │
 ├──confit.yml                    * the project config file generated by 'yo confit'
 ├──CONTRIBUTING.md               * how to contribute to the project
 ├──README.md                     * this file
 └──package.json                  * NPM package description file

GitFlow Development Process

This project uses the GitHub Flow workflow.

Create a branch

When you're working on a project, you're going to have a bunch of different features or ideas in progress at any given time – some of which are ready to go, and others which are not. Branching exists to help you manage this workflow.

When you create a branch in your project, you're creating an environment where you can try out new ideas. Changes you make on a branch don't affect the master branch, so you're free to experiment and commit changes, safe in the knowledge that your branch won't be merged until it's ready to be reviewed by someone you're collaborating with.

###ProTip

Branching is a core concept in Git, and the entire GitHub Flow is based upon it. There's only one rule: anything in the master branch is always deployable.

Because of this, it's extremely important that your new branch is created off of master when working on a feature or a fix. Your branch name should be descriptive (e.g., refactor-authentication, user-content-cache-key, make-retina-avatars), so that others can see what is being worked on.

Add commits

Once your branch has been created, it's time to start making changes. Whenever you add, edit, or delete a file, you're making a commit, and adding them to your branch. This process of adding commits keeps track of your progress as you work on a feature branch.

Commits also create a transparent history of your work that others can follow to understand what you've done and why. Each commit has an associated commit message, which is a description explaining why a particular change was made. Furthermore, each commit is considered a separate unit of change. This lets you roll back changes if a bug is found, or if you decide to head in a different direction.

###ProTip

Commit messages are important, especially since Git tracks your changes and then displays them as commits once they're pushed to the server. By writing clear commit messages, you can make it easier for other people to follow along and provide feedback.

Open a pull request

Pull Requests initiate discussion about your commits. Because they're tightly integrated with the underlying Git repository, anyone can see exactly what changes would be merged if they accept your request.

You can open a Pull Request at any point during the development process: when you have little or no code but want to share some screenshots or general ideas, when you're stuck and need help or advice, or when you're ready for someone to review your work. By using GitHub's @mention system in your Pull Request message, you can ask for feedback from specific people or teams, whether they're down the hall or ten time zones away.

###ProTip

Pull Requests are useful for contributing to open source projects and for managing changes to shared repositories. If you're using a Fork & Pull Model, Pull Requests provide a way to notify project maintainers about the changes you'd like them to consider. If you're using a Shared Repository Model, Pull Requests help start code review and conversation about proposed changes before they're merged into the master branch.

Discuss and review your code

Once a Pull Request has been opened, the person or team reviewing your changes may have questions or comments. Perhaps the coding style doesn't match project guidelines, the change is missing unit tests, or maybe everything looks great and props are in order. Pull Requests are designed to encourage and capture this type of conversation.

You can also continue to push to your branch in light of discussion and feedback about your commits. If someone comments that you forgot to do something or if there is a bug in the code, you can fix it in your branch and push up the change. GitHub will show your new commits and any additional feedback you may receive in the unified Pull Request view.

###ProTip

Pull Request comments are written in Markdown, so you can embed images and emoji, use pre-formatted text blocks, and other lightweight formatting.

Merge to master

Once your PR has passed any the integration tests and received approval to merge, it is time to merge your code into the master branch.

Once merged, Pull Requests preserve a record of the historical changes to your code. Because they're searchable, they let anyone go back in time to understand why and how a decision was made.

###ProTip

By incorporating certain keywords into the text of your Pull Request, you can associate issues with code. When your Pull Request is merged, the related issues are also closed. For example, entering the phrase Closes #32 would close issue number 32 in the repository. For more information, check out our help article.

Build Tasks

Command	Description

npm run build

| Generate production build into [dist/](dist/) folder

npm run dev

| Run project in development mode (verify code, and re-verify when code is changed)

npm start

| Alias for `npm run dev` task

Test Tasks

Command	Description

npm test

| Alias for `npm run test:unit` task

npm run test:coverage

| Run instrumented unit tests then verify coverage meets defined thresholds

• Returns non-zero exit code when coverage does not meet thresholds (as defined in istanbul.js)

npm run test:unit

| Run unit tests whenever JS source or tests change

• Uses Mocha
• Code coverage
• Runs continuously (best to run in a separate window)

npm run test:unit:once

| Run unit tests once

• Uses Mocha
• Code coverage

Verification (Linting) Tasks

Command	Description

npm run verify

| Verify code style and syntax

• Verifies source *and test code* aginst customisable rules (unlike Webpack loaders)

npm run verify:js

| Verify Javascript code style and syntax

npm run verify:js:fix

| Verify Javascript code style and syntax and fix any errors that can be fixed automatically

npm run verify:js:watch

| Verify Javascript code style and syntax and watch files for changes

npm run verify:watch

| Runs verify task whenever JS or CSS code is changed

Commit Tasks

Command	Description

git status

| Lists the current branch and the status of changed files

git log

| Displays the commit log (press Q to quit viewing)

git add .

| Stages all modified & untracked files, ready to be committed

git cz

| Commit changes to local repository using Commitizen.

• Asks questions about the change to generate a valid conventional commit message
• Can be customised by modifying [config/release/commitMessageConfig.js](config/release/commitMessageConfig.js)

git push

| Push local repository changes to remote repository

Release Tasks

Command	Description

npm run commitmsg

| Git commit message hook that validates the commit message conforms to your commit message conventions.

npm run release

| Verify code, run unit tests, check test coverage, build software.

• Define another script-task called `postrelease` to define the rest of your release process

Changing build-tool configuration

There are 3 ways you can change the build-tool configuration for this project:

1. BEST: Modify the Confit configuration file (confit.yml) by hand, then re-run yo confit and tell it to use the existing configuration.
2. OK: Re-run yo confit and provide new answers to the questions. Confit will attempt to overwrite your existing configuration (it will prompt for confirmation), so make sure you have committed your code to a source control (e.g. git) first. There are certain configuration settings which can only be specified by hand, in which case the first approach is still best.
3. RISKY: Modify the generated build-tool config by hand. Be aware that if you re-run yo confit it will attempt to overwrite your changes. So commit your changes to source control first.

Additionally, the currently-generated configuration can be extended in the following ways:

• The task configuration is defined in package.json. It is possible to change the task definitions to add your own sub-tasks. You can also use the pre... and post... script-name prefixes to run commands before (pre) and after (post) the generated commands.

• The entryPoint.entryPoints string in confit.yml is designed to be edited manually. It represents the starting-point(s) of the application (like a main() function). A NodeJS application has one entry point. E.g. src/index.js