Skip to content

feat(nestjs-fastify-graphql): init nestjs project with fastify and graphql#101

Merged
lamngockhuong merged 1 commit intomainfrom
task.init-nestjs-fastify-graphql-project
Oct 15, 2025
Merged

feat(nestjs-fastify-graphql): init nestjs project with fastify and graphql#101
lamngockhuong merged 1 commit intomainfrom
task.init-nestjs-fastify-graphql-project

Conversation

@hatruong219
Copy link
Copy Markdown
Contributor

Summary

Description

Checklist

  • 1. Build successfully on local
  • 2. Format all code files included in the PR
  • 3. No more unconfirmed specs in the PR
  • 4. Tested the normal case and error cases following specs
  • 5. Is there a refactor to the old code? If yes, have you tested the old function and noted the impact yet?
  • 6. Loops have a set length and correct termination conditions
  • 7. There aren't any redundant or duplicate codes in PR (clean code)
  • 8. There aren't any hard-coded in the code in PR
  • 9. Optimization logic handling and optimizations SQL (for example, don't get redundant data or add an index to optimize performance, query only necessary fields...)
  • 10. Update issue

Evidence

(Screenshot or Video)

Discussion

Note the part that you are not sure about. Comment on your code is OK.

@lamngockhuong lamngockhuong changed the title feat(#1): init nestjs project with fastify and graphql feat(nestjs-fastify-graphql): init nestjs project with fastify and graphql Oct 15, 2025
Copy link
Copy Markdown
Contributor

@lamngockhuong lamngockhuong left a comment

Choose a reason for hiding this comment

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

LGTM

@lamngockhuong lamngockhuong force-pushed the task.init-nestjs-fastify-graphql-project branch from f280dcc to 8401b12 Compare October 15, 2025 10:06
@lamngockhuong lamngockhuong merged commit b4f30f7 into main Oct 15, 2025
1 check passed
@lamngockhuong lamngockhuong deleted the task.init-nestjs-fastify-graphql-project branch October 15, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants