Pierre Beitz

Results 16 comments of Pierre Beitz

@timja I followed your advice and modify the `getLatestPluginVersion` directly. Not sure if this class is providing a public API for downstream projects so I kept the old method, let...

@oleg-nenashev good point but I'm not sure how the case you mention could happen given that if C is not proposed by the UC you will never be able to...

> if it's optional we can ignore it, not sure if you can set the version number to be 'optional'? Not sure I didn't read the whole codeline (yet :))...

@oleg-nenashev happy to improve this PR to get it over the finish line, would you mind giving me an example of what this PR would break, I'm not sure I...

Hey, thanks a lot for this PR, this helped a lot. However I had to add `USE_ID_TOKEN=true` for the setup to properly work (as per https://docs.influxdata.com/chronograf/v1/administration/managing-security/#jwks-signature-verification-optional).

Thanks! Most of the conflicts you'll see are due to the introduction of spotless on the codeline.

Sorry for the delay @raducoravu. According to the [documentation](https://developer.zendesk.com/api-reference/help_center/help-center-api/articles/#list-articles): > {/locale} is required only for end users or anonymous users. Admins and agents can omit it. So most likely, you...

Unsure I understand, do you need me to add the two endpoints in #701 so that end users/anonymous users can use the API, or are you telling me only admins...

Great, thanks for the feedback, I'll merge the PR adding the two methods.

Sorry, I'd rather not add lombok to this project. Feel free to provide a PR with builders though.