Marvin

Results 79 comments of Marvin

Sorry @joa, I don't touch flexmojos code since 2012, I don't remember it at all @silmerusse For workaround I suggest you creating a constructor.

It can be off by default...

True... although even on JPA, you are programming in annotations... we just assume not testing is ok... Another case where mutation would be really useful is on JSR-303 (bean validation)......

Hi @prashanth057 , feign is mainly run by volunteers. Either you need to get a volunteer interested to do it for you, or you file a PR for this and...

Sounds awesome, I would love to see this done. Feel free to ping me once you have a PR. Maybe, call it `@Shared` instead of configuration

Problem is if `EnvironmentExpander` needs to read some configuration from a database or need some other classes access.... Let's say I have a client that I authenticate and I get...

Another thing we need too consider is: Do we want this to affect ALL contracts or just the default contract?!

Wondering if we should do something like this instead: ``` var client = Feign.builder() .decoder(...) .encoder(...) .contract(...) .configuration("env", "staging") .target(...); ```

I like this: ``` Feign.builder() .decoder() .encoder() .contract() .parameter(name, new Expander()) // singleton dynamic use case .parameter(name, "static value") // constant use case .build() ``` I just don't see the...

I don't see any point on having the `@Param` if we include this on the builder. Am I missing something?