-
Notifications
You must be signed in to change notification settings - Fork 2
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
Optimize formatting and data loading in resolvers #28
Conversation
The primary action taken in this commit was the optimization of GraphQL resolvers. This was done by including a the 'formatUserForGraphql' function in issues, projects, and issue comments resolvers which helped streamline the process of formatting users for GraphQL. In addition, the DataLoader was primed with 'findIssueComment', improving data loading efficiency. Lastly, code related to GraphQL after file write hooks and mappers were revised in 'codegen.ts'.
PR Analysis
PR Feedback💡 General suggestions: The PR seems well-structured and focuses on optimization and code cleanup, which is good. However, it's important to ensure that the new utility function 'formatUserForGraphql' is thoroughly tested in all scenarios where it's used to prevent any potential regressions. Additionally, the use of 'sed' in 'codegen.ts' should be cross-platform compatible, as the current implementation may not work on all operating systems. 🤖 Code feedback:
✨ Usage guide:Overview:
With a configuration file, use the following template:
See the review usage page for a comprehensive guide on using this tool. |
The primary action taken in this commit was the optimization of GraphQL resolvers. This was done by including a the 'formatUserForGraphql' function in issues, projects, and issue comments resolvers which helped streamline the process of formatting users for GraphQL. In addition, the DataLoader was primed with 'findIssueComment', improving data loading efficiency. Lastly, code related to GraphQL after file write hooks and mappers were revised in 'codegen.ts'.