Skip to content

docs: Add GitHub links to Type Providers + mention Zod#5365

Merged
Uzlopak merged 11 commits intofastify:mainfrom
melroy89:patch-2
Mar 23, 2024
Merged

docs: Add GitHub links to Type Providers + mention Zod#5365
Uzlopak merged 11 commits intofastify:mainfrom
melroy89:patch-2

Conversation

@melroy89
Copy link
Copy Markdown
Contributor

@melroy89 melroy89 commented Mar 19, 2024 •
edited
Loading

  • Add links behind each package name, so people can easily find and look-up the corresponding READMEs
  • Add Zod to the TypeScript list as 3rd party
  • Add Type Provider wrapper packages to the Type Provider page
  • Make all links consistency, not using "GitHub" anymore

Checklist

melroy89 added 2 commits March 19, 2024 01:10
@melroy89
Add GitHub links and mention Zod
6d13ac3 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89
Fix text for TypeBox
adb170e 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
mcollina
mcollina approved these changes Mar 19, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

Fdawgs
Fdawgs requested changes Mar 19, 2024
View reviewed changes
melroy89 added 3 commits March 19, 2024 16:43
@melroy89
Merge branch 'main' into patch-2
4304d07
@melroy89
Create links around the package names
cf5fffc 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89
Also list the wrapper packages on Type Provider page + consistency
f9f7e30 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89 melroy89 requested a review from Fdawgs March 19, 2024 16:38
mcollina
mcollina approved these changes Mar 20, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

Fdawgs
Fdawgs requested changes Mar 21, 2024
View reviewed changes
melroy89 and others added 4 commits March 21, 2024 13:05
@melroy89 @Fdawgs
Update docs/Reference/TypeScript.md
7247e7e 
Co-authored-by: Frazer Smith <frazer.dev@outlook.com>
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89
Merge branch 'main' into patch-2
abcfb33
@melroy89
Add 3rd party
b0ec8d8 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89
Moved 3rs party line
8b09057 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89
Copy link
Copy Markdown
Contributor Author

melroy89 commented Mar 21, 2024

Owno I made the linter mad

melroy89 added 2 commits March 21, 2024 15:05
@melroy89
Fix linter?
f921f23 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89
Make linter happy, plz
b1f0385 
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89 melroy89 requested a review from Fdawgs March 21, 2024 14:15
mcollina
mcollina approved these changes Mar 21, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@melroy89
Copy link
Copy Markdown
Contributor Author

melroy89 commented Mar 22, 2024

Thanks for approving.

@Uzlopak Uzlopak merged commit 76674fd into fastify:main Mar 23, 2024
@melroy89 melroy89 deleted the patch-2 branch March 23, 2024 12:03
@urbankrepel urbankrepel mentioned this pull request May 28, 2024
Merged
@Viorel1989 Viorel1989 mentioned this pull request May 28, 2024
Merged
@raniellimontagna raniellimontagna mentioned this pull request May 28, 2024
Closed
@denbon05 denbon05 mentioned this pull request May 28, 2024
Merged
@X-oss-byte X-oss-byte mentioned this pull request May 29, 2024
Open
@JamieVangeysel JamieVangeysel mentioned this pull request May 29, 2024
Closed
@JamieVangeysel JamieVangeysel mentioned this pull request Jul 26, 2024
Closed
@JamieVangeysel JamieVangeysel mentioned this pull request Aug 2, 2024
Closed
@NicolasTOISON NicolasTOISON mentioned this pull request Aug 7, 2024
Merged
@JamieVangeysel JamieVangeysel mentioned this pull request Aug 11, 2024
Closed
This was referenced Aug 21, 2024
Open
Open
Open
Open
Open
Open
Open
@Jamesp918 Jamesp918 mentioned this pull request Aug 24, 2024
Open
@JamieVangeysel JamieVangeysel mentioned this pull request Aug 25, 2024
Closed
This was referenced Aug 28, 2024
Open
Open
This was referenced Sep 3, 2024
Closed
Closed
This was referenced Sep 8, 2024
Open
Open
@JamieVangeysel JamieVangeysel mentioned this pull request Sep 10, 2024
Closed
@WesleyR10 WesleyR10 mentioned this pull request Sep 12, 2024
Open
@JamieVangeysel JamieVangeysel mentioned this pull request Sep 18, 2024
Closed
@Jamesp918 Jamesp918 mentioned this pull request Sep 18, 2024
Open
@JamieVangeysel JamieVangeysel mentioned this pull request Sep 24, 2024
Closed
@Jamesp918 Jamesp918 mentioned this pull request Sep 25, 2024
Open
@JamieVangeysel JamieVangeysel mentioned this pull request Sep 27, 2024
Closed
@Jamesp918 Jamesp918 mentioned this pull request Oct 15, 2024
Open
@JamieVangeysel JamieVangeysel mentioned this pull request Oct 27, 2024
Closed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 24, 2025

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@mcollina mcollina mcollina approved these changes

@jsumners jsumners jsumners approved these changes

@Fdawgs Fdawgs Fdawgs approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@melroy89 @mcollina @jsumners @Fdawgs @Uzlopak