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

Implementations for Ktor, Spring, and Play #55

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

GirlInPurple
Copy link

@GirlInPurple GirlInPurple commented Jan 2, 2025

Here is a (work-in-progress) implementation for the JVM-based HTTP server frameworks Ktor, Spring, and Play.
I am open to suggestions for more JVM-based frameworks; I ended up dropping Netty support due to time constraints.

Stuff that still needs to be done:

  • General documentation improvements
    • Within the README in the /jvm/ directory
    • Within all the exposed APIs (mostly in :common)
    • Within the repo's main README
    • I am strongly considering making another PR purely for updating and adding documentation across all implementations.
  • CI Tests for all 3 subprojects
    • Ktor
    • Spring
    • Play
  • Backend support for all three platforms (to be specific, here)
    • I kept having to spoof being an Express server while testing an eariler version, it might be best to add an "Unknown" category for situations like this.
  • Upload all 3 subprojects to Maven Central or some equivalent
    • I've had some issues with Jitpack in the past, especially for Kotlin-based multiproject repos, but if you can get it to work properly it's worth a shot.

Copy link

vercel bot commented Jan 2, 2025

@GirlInPurple is attempting to deploy a commit to the tomdraper's projects Team on Vercel.

A member of the Team first needs to authorize it.

@tom-draper
Copy link
Owner

Wow! This is fantastic. I'll do some testing as soon as I can. Great work

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