Skip to content

Latest commit

 

History

History
161 lines (116 loc) · 6.62 KB

CONTRIBUTING.md

File metadata and controls

161 lines (116 loc) · 6.62 KB

Contribute To Revel

This describes how developers may contribute to Revel.

Mission

Revel's mission is to provide a batteries-included framework for making large scale web application development as efficient and maintainable as possible.

The design should be configurable and modular so that it can grow with the developer. However, it should provide a wonderful un-boxing experience and default configuration that can woo new developers and make simple web apps straight forward. The framework should have an opinion about how to do all of the common tasks in web development to reduce unnecessary cognitive load.

How To Contribute

Join The Community

The first step to making Revel better is joining the community! You can find the community on:

Once you've joined, there are many ways to contribute to Revel:

  • Report bugs (via GitHub)
  • Answer questions of other community members (via Google Groups or IRC)
  • Give feedback on new feature discussions (via GitHub and Google Groups)
  • Propose your own ideas (via Google Groups or GitHub)

How Revel Is Developed

We have begun to formalize the development process by adopting pragmatic practices such as:

  • Developing on the develop branch
  • Merging develop branch to master branch in 6 week iterations
  • Tagging releases with MAJOR.MINOR syntax (e.g. v0.8) ** We may also tag MAJOR.MINOR.HOTFIX releases as needed (e.g. v0.8.1) to address urgent bugs. Such releases will not introduce or change functionality
  • Managing bugs, enhancements, features and release milestones via GitHub's Issue Tracker
  • Using feature branches to create pull requests
  • Discussing new features before hacking away at it

How To Correctly Fork

Go uses the repository URL to import packages, so forking and go geting the forked project will not work.

Instead, follow these steps:

  1. Install Revel locally
  2. Fork Revel on Github
  3. Add your fork as a git remote

Here is the command line:

$ go get github.com/robfig/revel                      # Install Revel
$ cd $GOPATH/src/github.com/robfig/revel              # Change directory to revel repo
$ git remote add fork [email protected]:$USER/revel.git  # Add your fork as a remote

Create A Feature Branch & Code Away!

Now that you've properly installed and forked Revel, you are ready to start coding (assuming you have a valdiated your ideas with other community members)!

In order to have your pull requests accepted, we recommend you make your changes to Revel on a new git branch. For example,

$ git checkout -b feature/useful-new-thing develop    # Create a new branch based on the develop branch and switch to it
$ ...                                                 # Make your changes and commit them
$ git pull origin develop                             # Optionally, merge new changes from upstream
$ git push fork develop                               # After new commits, push to your fork

Gofmt Your Code

Set your editor to run "go fmt" every time you save so that whitespace / style comments are kept to a minimum.

Howtos:

Write A Test (And Benchmark For Bonus Points)

Significant new features require tests. Besides unit tests, it is also possible to test a feature by exercising it in one of the sample apps and verifying its operation using that app's test suite. This has the added benefit of providing example code for developers to refer to.

Benchmarks are helpful but not required.

Run The Tests

Typically running the main set of unit tests will be sufficient:

$ go test github.com/robfig/revel

Refer to the Travis configuration for the full set of tests. They take less than a minute to run.

Document Your Feature

Due to the wide audience and shared nature of Revel, documentation is an essential addition to your new code. Pull requests risk not being accepted until proper documentation is created to detail how to make use of new functionality.

The Revel web site is hosted on Github-pages and built with Jekyll.

To develop the Jekyll site locally:

# Clone a second repository and check out the branch
$ git clone [email protected]:robfig/revel.git
$ cd revel
$ git checkout gh-pages

# Install / run Jekyll 1.0.3 to generate the site, and serve the result
$ gem install jekyll -v 1.0.3
$ jekyll build --watch --safe -d test/revel &
$ cd test
$ python -m SimpleHTTPServer 8088

# Now load in your browser
$ open http://localhost:8088/revel

Any changes you make to the site should be reflected within a few seconds.

Submit Pull Request

Once you've done all of the above & pushed your changes to your fork, you can create a pull request for review and acceptance.

Potential Projects

These are outstanding feature requests, roughly ordered by priority. Additionally, there are frequently smaller feature requests or items in the issues.

  1. Better ORM support. Provide more samples (or modules) and better documentation for setting up common situations like SQL database, Mongo, LevelDB, etc.
  2. Support for other templating languages (e.g. mustache, HAML). Make TemplateLoader pluggable. Use Pongo instead of vanilla Go templates (and update the samples)
  3. Test Fixtures
  4. Authenticity tokens for CSRF protection
  5. Coffeescript pre-processor. Could potentially use otto as a native Go method to compiling.
  6. SCSS/LESS pre-processor.
  7. GAE support. Some progress made in the 'appengine' branch -- the remaining piece is running the appengine services in development.
  8. More Form helpers (template funcs).
  9. A Mongo module (perhaps with a sample app)
  10. Easy emailer support (e.g. to email exception logs to developer, or even to email users),
  11. Deployment to OpenShift (support, documentation, etc)
  12. Improve the logging situation. The configuration is a little awkward and not very powerful. Integrating something more powerful would be good. (like seelog or log4go)
  13. ETags, cache controls
  14. A module or plugins for adding HTTP Basic Auth
  15. Allowing the app to hook into the source code processing step