Tom McLaughlin

Results 191 comments of Tom McLaughlin

Thanks for the quick response! FWIW, I have read that while there isn't good universal agreement about how plus signs in URLs should be treated, the [W3C URI spec](https://www.w3.org/Addressing/URL/uri-spec.html) says...

Yes @hughjfchen, I referenced the relevant commit (541fb764666f84acff1ecb6c480365f0f3e79a64) in the initial post. I'd say "fixed" is a strong word :P

Okay, so if I understand correctly the `openapi-generator` has not been updated to deal with later versions of `hoauth2`, right? So if we regenerate right now, the result won't build...

Thanks for updating the generator! I built it locally with both Aeson 1 and 2 and it looks good. Trying to update the Travis config now. Have you considered switching...

Okay, like this? https://github.com/kubernetes/test-infra/pull/26444

@jonschoning maybe it's time to give up on https://github.com/kubernetes/test-infra/pull/26444 ? Is it possible to merge this anyway?

@brendandburns could we get a trusted user review on this minor change? Thanks!

@brendandburns friendly ping, would still like to move this along if possible. Thanks!

FWIW, pandoc is a huge and also GPL-ed dependency. I suspect something homemade like Microsoft did is the cleanest way to do it. Or maybe an existing Python library like...

@michaelpj it seems to be a direct dependency: ``` ╚═══bin/haskell-nix-closure-exe: ….includes/rts./nix_frozen/store/ly011abgbh1n2w63rqxa98j1n6xa8zd4-gcc-9.2.0/lib/gcc/x86_64-unknow… => /nix_frozen/store/ly011abgbh1n2w63rqxa98j1n6xa8zd4-gcc-9.2.0 ``` There's some mention of `includes/rts` so maybe RTS related?