Skip to content

Commit

Permalink
Merge pull request #51 from maclover7/contributing
Browse files Browse the repository at this point in the history
Add CONTRIBUTING.md [ci skip]
  • Loading branch information
rafaelfranca committed Sep 12, 2015
2 parents 398ea81 + dce57f3 commit 19da525
Show file tree
Hide file tree
Showing 3 changed files with 109 additions and 1 deletion.
99 changes: 99 additions & 0 deletions CONTRIBUTING.md
@@ -0,0 +1,99 @@
Contributing to Active Record Session Store
=====================

[![Build Status](https://travis-ci.org/rails/activerecord-session_store.svg?branch=master)](https://travis-ci.org/rails/activerecord-session_store)

Active Record Session Store is work of [many contributors](https://github.com/rails/activerecord-session_store/graphs/contributors). You're encouraged to submit [pull requests](https://github.com/rails/activerecord-session_store/pulls), [propose features and discuss issues](https://github.com/rails/activerecord-session_store/issues).

#### Fork the Project

Fork the [project on Github](https://github.com/rails/activerecord-session_store) and check out your copy.

```
git clone https://github.com/contributor/activerecord-session_store.git
cd activerecord-session_store
git remote add upstream https://github.com/rails/activerecord-session_store.git
```

#### Create a Topic Branch

Make sure your fork is up-to-date and create a topic branch for your feature or bug fix.

```
git checkout master
git pull upstream master
git checkout -b my-feature-branch
```

#### Bundle Install and Test

Ensure that you can build the project and run tests.

```
bundle install
bundle exec rake test
```

#### 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.

```
git add ...
git commit
```

#### Push

```
git push origin my-feature-branch
```

#### Make a Pull Request

Go to https://github.com/contributor/activerecord-session_store 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.
2 changes: 1 addition & 1 deletion LICENSE → MIT-LICENSE
@@ -1,4 +1,4 @@
Copyright (c) 2012 David Heinemeier Hansson
Copyright (c) 2012-2015 David Heinemeier Hansson

MIT License

Expand Down
9 changes: 9 additions & 0 deletions README.md
Expand Up @@ -86,3 +86,12 @@ MyLogger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer

This silencer is being used to silence the logger and not leaking private
information into the log, and it is required for security reason.

## Contributing to Active Record Session Store

Active Record Session Store is work of many contributors. You're encouraged to submit pull requests, propose features and discuss issues.

See [CONTRIBUTING](CONTRIBUTING.md).

## License
Active Record Session Store is released under the [MIT License](MIT-LICENSE).

0 comments on commit 19da525

Please sign in to comment.