Fork me on GitHub
#ring-swagger
<
2024-04-01
>
RAJKUMAR04:04:31

@ikitommi I have come up with support for openapi 3.0

RAJKUMAR04:04:45

please let me know whether I can create merge request

ambrosebs05:04:48

@U02PR896TMG please make a pull request, I'm a maintainer and I will review it.

☝️ 1
RAJKUMAR14:04:52

Thank you so much @U055XFK8V!!!

RAJKUMAR14:04:21

I created the PR https://github.com/metosin/ring-swagger/pull/150. I need to add more tests but I would like to get feedback before that

❤️ 1
ambrosebs15:04:07

I gave the first round of superficial feedback, I will try it out at work.

RAJKUMAR03:04:44

to try it out in repl use these snippets

RAJKUMAR03:04:15

@U055XFK8V Any Idea about error in the build?

ambrosebs04:04:00

I'll look tomorrow.

ambrosebs04:04:43

BTW I just got funding to work on ring-swagger. You might want to brace yourself about whether to laugh or cry about what I proposed to accomplish 😉 https://www.clojuriststogether.org/news/q2-2024-funding-announcement/

wow 1
ikitommi04:04:30

Congratulations!!

❤️ 1
RAJKUMAR05:04:18

congratulations 🙂

RAJKUMAR14:04:39

apart from the joke, please let me know you are able to test the pr locally for openapi 3.0 or any questions

RAJKUMAR05:04:54

@U055XFK8V I updated the code for your main comment

RAJKUMAR05:04:02

could you please take another look

RAJKUMAR23:04:27

@U055XFK8V I see you pushed some commits to this PR

RAJKUMAR23:04:54

one question

RAJKUMAR23:04:23

When can the ring-swagger with open-api support be possibly released

RAJKUMAR23:04:13

one of my project needs this so would like to know

ambrosebs00:04:03

I have no idea sorry. It will be done at the latest by the time my funding is over July 1st.

ambrosebs00:04:39

LMK if it's better for me to work on a separate branch, I don't want to break anything you're depending on.

RAJKUMAR18:04:51

I think it is fine I can pull the latest changes

👍 1
ambrosebs19:05:07

@U02PR896TMG I had to rename all your namespaces FYI so be careful when pulling.

RAJKUMAR21:05:00

ohh okay thanks for letting me know 👍 @U055XFK8V