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

doc: RuntimeAdapter #87

Merged
merged 1 commit into from
Dec 11, 2024
Merged

doc: RuntimeAdapter #87

merged 1 commit into from
Dec 11, 2024

Conversation

magne4000
Copy link
Owner

No description provided.

@magne4000 magne4000 merged commit d0a2f49 into main Dec 11, 2024
6 checks passed
@magne4000 magne4000 deleted the magne4000/doc-runtime-adapter branch December 11, 2024 13:08
@magne4000
Copy link
Owner Author

@brillout regarding our discussion vikejs/vike-node#26 (comment)

I published some doc around RuntimeAdapter that I merged for convenience https://universal-middleware.dev/reference/runtime-adapter

Do you think it's enough and understandable if we link this page from vike-node README?

@brillout
Copy link

LGTM, awesome. I'll leave a handful of comments about polishing potential.

@brillout
Copy link

Little thing in case it's easy to change: the inline code blocks have a blue text so I thought there were links, but there aren't links.

@brillout
Copy link

Review done. Really neat reference page.

@magne4000
Copy link
Owner Author

Thanks for the review 👏

Little thing in case it's easy to change: the inline code blocks have a blue text so I thought there were links, but there aren't links.

Now that you mention it, it feels weird. It should be easy to change

@brillout
Copy link

Glad to help 💫

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