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

feat(type-safe-api): remove wrapper scripts for websocket docs projects #861

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

cogwirrel
Copy link
Member

Remove the wrappers around the asyncapi documentation generators and instead call them directly with npx.

Remove the wrapper around generating the asyncapi spec and instead add it as a subcommand of the type-safe-api script to make it much faster.

NB this removes the test for markdown asyncapi docs generation as it was more there to test the wrapper script rather than the behaviour of the dependency we use to generate the docs.

Remove the wrappers around the asyncapi documentation generators and instead call them
directly with `npx`.

Remove the wrapper around generating the asyncapi spec and instead add it as a subcommand of the
type-safe-api script to make it much faster.
Copy link

nx-cloud bot commented Oct 17, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit e3b3b7e. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@cogwirrel cogwirrel merged commit 174a093 into mainline Oct 17, 2024
4 checks passed
@cogwirrel cogwirrel deleted the feat/asyncapi-wrappers branch October 17, 2024 23:59
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