Add the compact mode

[jalkoby]

Add the compact option which strips useless _path suffixes in path helper functions. It should be useful in a situations when a "Routes" object has only path functions, so instead repeat again and again

Routes.users_path()
// it will become
Routes.users()

[le0pard]

Hello, @jalkoby I don't know how right now, but this changes broke tests with rails 3.2 (looks like segmentation fault) :)

[le0pard]

in this case better add for suffix default value nil

[jalkoby]

why do we need a default value, if all usages of this methods pass all arguments?

[le0pard]

Just be sure about (:path unless @options[:compact]). But you can leave as is. Just need fix problem with rails 3.2.

[bogdan]

I'd not use this option personally: the ability to grep particular route and see its usages is more important to me than avoid "useless" _path.

So, On your place I would think twice before turning it on for your app.

[jalkoby]

@bogdan I don't see any potential problem to find a places where some route helper function is used because all route helper functions are stored into Router object. So, grep "Router.someInvalidRoute" will be enough for 95% use-cases.You see, Rails adds "_path" and "_url" suffix only because it has a lot of other helpers("_tag", "options", "number" families) and it's really hard to determine what is a responsibility of a helper method. But helper methods which are located into an object with a single responsibility don't need useless suffix(for example, rails form builder which has methods text_area, text_field, select, but not a text_area_tag, text_field_tag, etc)

[bogdan]

Ok, I am fine with this option. Lets fix the build and we will merge it.

[jalkoby]

The problem in rails 3.2 env is related with errors in a C code. After a short review it seems libv8 was updated from time of the last stable build, so I've locked libv8 gem for 3.2 env

[le0pard]

@jalkoby need to fix this for jruby:

platforms :ruby do
  gem "libv8", "<= 3.16.14.3"
end

[le0pard]

@jalkoby All looks good. Just need add some descriptions about new option in README.

[jalkoby]

Sorry guys for the delay. I've updated Readme and combined all changes into single commit

[bogdan]

In the example fo previous option Example is used a example value of an option. So I would using example here.
And write something like: Sample route call when option is set to true:.