Skip to content

Latest commit

 

History

History
61 lines (42 loc) · 1.93 KB

CONTRIBUTING.md

File metadata and controls

61 lines (42 loc) · 1.93 KB
Raw

Contributing to re-frame-async-flow-fx

Thank you for taking the time to contribute!

Support questions

The Github issues are for bug reports and feature requests only. Support requests and usage questions should go to the re-frame Clojure Slack channel or the ClojureScript mailing list.

Pull requests

Create pull requests to the master branch, work will be merged when it is ready to be released.

Running tests

Via Browser/HTML

To build the tests and run them in one step, just:

lein test-once  # compiles & then opens test.html in the browser

You can also get auto compiles via:

lein test-auto

but you'll need to manually open test/test.html in a browser. And you'll also need to manually reload this page after each auto compile.

Via Karma

To run the tests, you must have recent versions of node, npm, Leiningen, and a C++ compiler toolchain installed. If you're on Linux or Mac OS X then you will be fine, if you're on Windows then you need to install Visual Studio Community Edition, and the C++ compiler dependencies.

lein deps       # runs lein-npm, installs Karma & other node dependencies. Only needed the first time.
lein karma-once # to build re-frame-async-flow-fx tests
karma start     # to run the tests with an auto watcher

Pull requests for bugs

If possible provide:

  • Code that fixes the bug.
  • Failing tests which pass with the new changes.
  • Improvements to documentation to make it less likely that others will run into issues (if relevant).
  • Add the change to the Unreleased section of CHANGELOG.md

Pull requests for features

If possible provide:

  • Code that implements the new feature
  • Tests to cover the new feature including all of the code paths
  • Docstrings for functions
  • Documentation examples
  • Add the change to the Unreleased section of CHANGELOG.md