Skip to content

Latest commit

 

History

History
183 lines (118 loc) · 5.38 KB

CONTRIBUTING.md

File metadata and controls

183 lines (118 loc) · 5.38 KB
Raw

Contributing to NGXS

👍🎉 First off, thanks for taking the time to contribute! 🎉👍

The following is a set of guidelines for contributing to NGXS and its packages.

Table Of Contents

I don't want to read this whole thing, I just have a question!!!

How can I help?

Developing

Building

Publish new version

Commit Message Guidelines

I don't want to read this whole thing I just have a question!!!

Note: Please don't file an issue to ask a question.

How can I help?

Check the issues where we have labels for help wanted. Also chat us up on slack, where we will be more than happy to get you started on an issue.

Developing

Start by installing all dependencies

yarn

You can then either start develping using TDD

yarn test

Or start the integration app and start playing around

yarn start

Building

Since our library is a combination of the main @ngxs/store and the plugins, we need to build them into separate npm packages and publish them.

running

yarn build

will create a build for each package in the builds/ directory.

Linking

If you want to test out a local build in your own app you can create a global symlink for the package which can then be linked locally in your app.

# build ngxs
yarn build

# cd into the build you want to link
cd builds/store

# create the global symlink
yarn link

now navigate to your app then run

yarn link @ngxs/store

now node_modules/@ngxs/store will be symlinked to the local build directory of @ngxs/store. to make a change, save the file in ngxs, run yarn build and your app should now be using the new build.

Publishing new version

(@ngxs team members only)

Dev builds

CircleCI picks up all commits to master, which is our development branch. It then runs the build, the tests and the integration tests. If they all succeed, CircleCI will publish to Code Climate, and to npm using the git commit sha1 as an identified 3.0.0-dev.6e59e7b and tag it as @dev

Beta builds

@todo need to figure out how to get the prerelease version and use it as the git tag, before we can automate the @beta releases.

Release builds

CircleCI picks up when a git tag is created and does the same thing as for the development build, but instead of releasing a dev build, releases the version that was specified in the root package.json in the git tag. It also tags it as @latest.

Commit Message Guidelines

We have very precise rules over how our git commit messages can be formatted. This leads to more readable messages that are easy to follow when looking through the project history. But also, we use the git commit messages to generate the Angular change log.

Commit Message Format

Each commit message consists of a header, a body and a footer. The header has a special format that includes a type, a scope and a subject:

<type>(<scope>): <subject>
<BLANK LINE>
<body>
<BLANK LINE>
<footer>

The header is mandatory and the scope of the header is optional.

Any line of the commit message cannot be longer 100 characters! This allows the message to be easier to read on GitHub as well as in various git tools.

The footer should contain a closing reference to an issue if any.

Samples: (even more samples)

docs(changelog): update changelog to beta.5

fix(release): need to depend on latest rxjs

The version in our package.json gets copied to the one we publish, and users need the latest of these.

Revert

If the commit reverts a previous commit, it should begin with revert:, followed by the header of the reverted commit. In the body it should say: This reverts commit <hash>., where the hash is the SHA of the commit being reverted.

Type

Must be one of the following:

  • 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

Scope

The scope should be the name of the npm package affected (as perceived by the person reading the changelog generated from commit messages.

The following is the list of supported scopes:

  • devtools-plugin
  • form-plugin
  • logger-plugin
  • router-plugin
  • storage-plugin
  • websocket-plugin
  • store
  • cli