1
0
Fork 0
mirror of https://github.com/rails/execjs synced 2023-03-27 23:21:20 -04:00
execjs/CONTRIBUTING.md

109 lines
2.9 KiB
Markdown
Raw Normal View History

2015-09-01 11:29:15 -04:00
Contributing to ExecJS
=====================
2014-05-19 09:59:29 -04:00
2015-09-01 11:29:15 -04:00
[![Build Status](https://travis-ci.org/rails/execjs.svg?branch=master)](https://travis-ci.org/rails/execjs)
2014-05-19 09:59:29 -04:00
2015-09-01 11:29:15 -04:00
ExecJS is work of [many contributors](https://github.com/rails/execjs/graphs/contributors). You're encouraged to submit [pull requests](https://github.com/rails/execjs/pulls), [propose features and discuss issues](https://github.com/rails/execjs/issues).
#### Fork the Project
Fork the [project on Github](https://github.com/rails/execjs) and check out your copy.
2014-05-19 09:59:29 -04:00
```
2015-09-01 11:29:15 -04:00
git clone https://github.com/contributor/execjs.git
cd execjs
git remote add upstream https://github.com/rails/execjs.git
2014-05-19 09:59:29 -04:00
```
2015-09-01 11:29:15 -04:00
#### Create a Topic Branch
Make sure your fork is up-to-date and create a topic branch for your feature or bug fix.
2014-05-19 09:59:29 -04:00
```
2015-09-01 11:29:15 -04:00
git checkout master
git pull upstream master
git checkout -b my-feature-branch
2014-05-19 09:59:29 -04:00
```
2015-09-01 11:29:15 -04:00
#### Bundle Install and Test
2014-05-19 09:59:29 -04:00
2015-09-01 11:29:15 -04:00
Ensure that you can build the project and run tests.
2014-05-19 09:59:29 -04:00
```
2015-09-01 11:29:15 -04:00
bundle install
bundle exec rake
```
Note that test:miniracer is failed in the tests, because mini_racer is not installed, and it is mutually exclusive with therubyracer.
To run test:miniracer
```
EXECJS_RUNTIME=MiniRacer bundle install --path=<path>
EXECJS_RUNTIME=MiniRacer bundle exec rake test:miniracer
```
2015-09-01 11:29:15 -04:00
#### Write Tests
Try to write a test that reproduces the problem you're trying to fix or describes a feature that you want to build. Add to [test](test).
We definitely appreciate pull requests that highlight or reproduce a problem, even without a fix.
#### Write Code
Implement your feature or bug fix.
Make sure that `bundle exec rake test` completes without errors.
#### Write Documentation
Document any external behavior in the [README](README.md).
#### Commit Changes
Make sure git knows your name and email address:
```
git config --global user.name "Your Name"
git config --global user.email "contributor@example.com"
```
Writing good commit logs is important. A commit log should describe what changed and why.
2014-05-19 09:59:29 -04:00
```
2015-09-01 11:29:15 -04:00
git add ...
git commit
```
#### Push
```
git push origin my-feature-branch
```
#### Make a Pull Request
Go to https://github.com/contributor/execjs and select your feature branch. Click the 'Pull Request' button and fill out the form. Pull requests are usually reviewed within a few days.
#### Rebase
If you've been working on a change for a while, rebase with upstream/master.
```
git fetch upstream
git rebase upstream/master
git push origin my-feature-branch -f
```
#### Check on Your Pull Request
Go back to your pull request after a few minutes and see whether it passed muster with Travis-CI. Everything should look green, otherwise fix issues and amend your commit as described above.
#### Be Patient
It's likely that your change will not be merged and that the nitpicky maintainers will ask you to do more, or fix seemingly benign problems. Hang on there!
#### Thank You
Please do know that we really appreciate and value your time and work. We love you, really.