-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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(docs): Add nest/terminus documentation #198
feat(docs): Add nest/terminus documentation #198
Conversation
b9885cd
to
16cecd4
Compare
src/app/homepage/pages/recipes/terminus/terminus.component.html
Outdated
Show resolved
Hide resolved
src/app/homepage/pages/recipes/terminus/terminus.component.html
Outdated
Show resolved
Hide resolved
src/app/homepage/pages/recipes/terminus/terminus.component.html
Outdated
Show resolved
Hide resolved
<span class="filename">app.module.ts</span> | ||
<pre><code class="language-typescript">{{ terminusOptionsServiceDog }}</code></pre> | ||
<p> | ||
If everything done correctly, the response of the server when request the <code>/health</code> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reword the sentence. Especially "the response of the server when request the [...]" is confusing to me as first time reader
daeb73c
to
4c7782b
Compare
Thanks a lot @weeco ! I adjusted and merged your suggestions |
Great job :) I'm gonna review that locally very soon |
4c7782b
to
2c8ad43
Compare
Can this be merged? @kamilmysliwiec |
Merged! Luckily, it was the last time when we had to create a chapter using HTML+props instead of markdown syntax. :) |
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: nestjs/terminus#5
What is the new behavior?
Added documentation for nestjs/termius
Does this PR introduce a breaking change?
Other information
Blockers: