Currently the type parameter representing the request context is spelled out as RequestCtx: RequestContext . This originally made (at least some) sense when we had multiple type parameters and we wanted to have some clarity (but now, there is only one type parameter).
I propose changing this simply to R: RequestContext. This will help in 2 ways. One, there will be less confusion on which is the type parameter (R) and which is the trait bound (RequestContext). Second, it will be less verbose, making the code a little bit more concise/cleaner.
@maldrake - thoughts?
Currently the type parameter representing the request context is spelled out as
RequestCtx: RequestContext. This originally made (at least some) sense when we had multiple type parameters and we wanted to have some clarity (but now, there is only one type parameter).I propose changing this simply to
R: RequestContext. This will help in 2 ways. One, there will be less confusion on which is the type parameter (R) and which is the trait bound (RequestContext). Second, it will be less verbose, making the code a little bit more concise/cleaner.@maldrake - thoughts?