Skip to content

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Oct 29, 2014

Normally I would submit a squashed commit, but I want to keep each item clear. @jodosha if you prefer I can split these into individual pull requests.

These changes are minor and trivial but appease the inch-ci monster.

http://inch-ci.org/github/lotus/controller
Lotus::Controller::Configuration#handle_exceptions=

> Suggestions:
> * Add a comment describing the method.
> * Describe what "handle_exceptions=" returns.
> * Add a code example (optional).

http://inch-ci.org/github/lotus/controller
Lotus::Controller::Configuration#load!

> Suggestions:
> * Describe what "load!" returns.
> * Describe the parameter "base".
> * Add a code example (optional).

http://inch-ci.org/github/lotus/controller
Lotus::Action::Cache#fresh

> Suggestions:
>
> * Describe what "fresh" returns.
> * Describe the parameter "options".

http://inch-ci.org/github/lotus/controller
Lotus::Action::Cache#expires

> * Describe what "expires" returns.
> * Describe the parameter "amount".
> * Describe the parameter "values".

http://inch-ci.org/github/lotus/controller
Lotus::Action::Cache#cache_control

> * Describe what "cache_control" returns.
> * Describe the parameter "values".

http://inch-ci.org/github/lotus/controller
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 376d878 on jeremyf:improving-documentation into d8b5bbd on lotus:master.

@jodosha jodosha self-assigned this Oct 29, 2014
@jodosha
Copy link
Member

jodosha commented Oct 29, 2014

@jeremyf It's good as it is, thank you!

jodosha added a commit that referenced this pull request Oct 29, 2014
@jodosha jodosha merged commit f1dd1a3 into hanami:master Oct 29, 2014
@jeremyf jeremyf deleted the improving-documentation branch October 29, 2014 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants