Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed missing dash in xiana.swagger #278

Closed
wants to merge 5 commits into from
Closed

Conversation

patbrown
Copy link

@patbrown patbrown commented Jan 8, 2024

  • get-apidoc is -get-apidoc

Title: Spelling error on -get-apidocs in xiana.swagger

Description

Says it all. There was a dash missing from -get-apidocs. If you want me to put something on the changelog I'll resubmit. I'll leave it up to you if you want to track such minor things.

Xiana

I'm enjoying how minimalistic what you are doing is. Thanks for the open source.

@soulflyer
Copy link
Contributor

soulflyer commented Jan 18, 2024

-get-apidocs refers to the Protocol but we are calling the function which is get-apidocs. Note that we are using version 0.5.18 not the most recent version which doesn't appear to have the fn, only the protocol. That is something we will need to sort out if and when we update the reitit version.

@@ -1,7 +1,7 @@
(ns xiana.webserver
"Lifecycle management of the webserver"
(:require
[ring.adapter.jetty9 :as jetty]
[org.httpkit.server :as server]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we want to change to httpkit?

@soulflyer
Copy link
Contributor

There appear to be a number of things happening in this PR. We really need to stick to one issue per PR.

@soulflyer soulflyer closed this Feb 1, 2024
@patbrown
Copy link
Author

patbrown commented Feb 7, 2024

Oh geez, please forgive me. I was late to see this. I really messed up this pull request. I only wanted to PR fixing the dash. SORRY!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants