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

[Maintenance] Updated next-auth-hasura-adapter dependencies #2

Merged

Conversation

agustints
Copy link
Contributor

Hi Amruth Pillai 👋

I hope this message finds you well. I have thoroughly enjoyed working on this project and have made several updates and refactors based on the recent commits. Here's a summary of the changes I made:

  • Updated the project's library dependencies to their latest compatible versions, enhancing security and leveraging new features.
    I believe these changes significantly improve the project's maintainability, code quality, and alignment with the latest best practices. I have thoroughly tested the modifications and ensured they integrate seamlessly into the existing codebase.

  • Added necessary TypeScript types to enhance code integrity and maintainability.

  • Included a Prettier configuration file to ensure consistent code formatting across the project.

  • Transitioned from YAML to TypeScript for codegen configuration due to major changes updates
    Migrated the code generation configuration from YAML to TypeScript to accommodate recent major changes and updates.

Thank you for your time and consideration. I look forward to your feedback and the opportunity to contribute further to this open-source project. Please let me know if there's anything else I can assist you with.

Best regards,
Agustin Tornielli

@AmruthPillai
Copy link
Owner

Sorry about the late response, I was on vacation. Back today. This is absolutely great, very clean changes. Thank you so much for your contribution! ❤️

@AmruthPillai AmruthPillai merged commit fb20d7c into AmruthPillai:main Jul 19, 2023
@AmruthPillai AmruthPillai mentioned this pull request Jul 20, 2023
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