<@UL5UQQ518> nope. I can just release it if you want. The only thing it hasn't got at the moment is ...

# http4k

[dave]

@Ricardo nope. I can just release it if you want. The only thing it hasn't got at the moment is the "include the docs route" PR.

[Ricardo]

I just wanted to double-check the new OpenAPI2 stuff produces valid Swagger before you folks release.

[dave]

well we do have tests.... 🙂

[dave]

releasing isn't a biggie, but if it's no good to you without the last piece then we should hold off

[Ricardo]

LOL you do, but they don’t test the OpenAPI stuff matches the schema. I think the only thing that does is an command line NPM tool.

[Ricardo]

Yeah, the last piece means we can get rid of a bunch of hacks in our build scripts.

[dave]

our build generates a client using the OpenApi2 generator

[dave]

(and OpenApi3)

[dave]

so if it IS wrong, the OpenApi generator is also wrong 🙂

[dave]

(there is a validation step)

[Ricardo]

Ah OK, cool, I didn’t know that.

[dave]

not a lot of people do 😉

[Ricardo]

That puts my mind to rest 🙂

[dave]

good -- shall I hold off then or release it?

[Ricardo]

For the description stuff? Yes please. Let me know if the PR needs any changes.

[dave]

sorry - am I holding off until that PR is in, or releasing now? 🙂

[Ricardo]

Sorry, if you could hold off please.

[dave]

@Ricardo 3.165.0 is now in bintray and working it's way to maven central. https://github.com/http4k/http4k/blob/master/CHANGELOG.md#changelog

[Ricardo]

Fantastic, thank you. I’ll upgrade on Monday when I’m in the office.

[Ricardo]

Integrated, tested and deployed. Worked great, thanks David!