Skip to content

Commit

Permalink
Allow path => controller#action in Routes
Browse files Browse the repository at this point in the history
The `'path' => 'controller#action'` is a supported short-hand syntax
for Rails routes as documented [here][1]. The current type is too
restrictive and does not support this valid syntax. This change modifies
the first argument type so that it supports the Hash syntax.

[1]: https://github.com/rails/rails/blob/master/actionpack/lib/action_dispatch/routing/mapper.rb#L1605
  • Loading branch information
panthomakos committed Aug 6, 2019
1 parent 27c2868 commit bc7e186
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions lib/actionpack/all/actionpack.rbi
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ module ActionDispatch::Routing::Mapper::HttpHelpers
# ActionDispatch::Routing::Mapper::Resources#match
sig do
params(
name: T.any(String, Symbol),
name: T.any(String, Symbol, T::Hash[String, String]),
controller: T.nilable(T.any(String, Symbol)),
action: T.nilable(T.any(String, Symbol)),
param: T.nilable(Symbol),
Expand Down Expand Up @@ -148,7 +148,7 @@ module ActionDispatch::Routing::Mapper::HttpHelpers

sig do
params(
name: T.any(String, Symbol),
name: T.any(String, Symbol, T::Hash[String, String]),
controller: T.nilable(T.any(String, Symbol)),
action: T.nilable(T.any(String, Symbol)),
param: T.nilable(Symbol),
Expand Down Expand Up @@ -185,7 +185,7 @@ module ActionDispatch::Routing::Mapper::HttpHelpers

sig do
params(
name: T.any(String, Symbol),
name: T.any(String, Symbol, T::Hash[String, String]),
controller: T.nilable(T.any(String, Symbol)),
action: T.nilable(T.any(String, Symbol)),
param: T.nilable(Symbol),
Expand Down Expand Up @@ -222,7 +222,7 @@ module ActionDispatch::Routing::Mapper::HttpHelpers

sig do
params(
name: T.any(String, Symbol),
name: T.any(String, Symbol, T::Hash[String, String]),
controller: T.nilable(T.any(String, Symbol)),
action: T.nilable(T.any(String, Symbol)),
param: T.nilable(Symbol),
Expand Down Expand Up @@ -259,7 +259,7 @@ module ActionDispatch::Routing::Mapper::HttpHelpers

sig do
params(
name: T.any(String, Symbol),
name: T.any(String, Symbol, T::Hash[String, String]),
controller: T.nilable(T.any(String, Symbol)),
action: T.nilable(T.any(String, Symbol)),
param: T.nilable(Symbol),
Expand Down Expand Up @@ -304,7 +304,7 @@ module ActionDispatch::Routing::Mapper::Resources

sig do
params(
name: T.any(String, Symbol),
name: T.any(String, Symbol, T::Hash[String, String]),
controller: T.nilable(T.any(String, Symbol)),
action: T.nilable(T.any(String, Symbol)),
param: T.nilable(Symbol),
Expand Down

0 comments on commit bc7e186

Please sign in to comment.