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

feat: updates generateTypeHelper to use type keyword and updates prettier #243

Merged
merged 1 commit into from
Feb 18, 2024

Conversation

JoA-MoS
Copy link
Contributor

@JoA-MoS JoA-MoS commented Feb 15, 2024

  • adds support for verbatimModuleSyntax
  • adds support for user-defined prettier config
  • updates prettier and related dependencies to the latest versions

BREAKING CHANGE: Typescript < 3.8 is no longer supported, please update to typescript >= 3.8

COMPLETES: #242

Files with functional changes unrelated to prettier write:

…tier version

* adds support for verbatimModuleSyntax
* adds support for user defined prettier config
* updates prettier and related dependencies to latest versions

BREAKING CHANGE: Typescript < 3.8 is no longer supported, please update to typescript >= 3.8

COMPLETES: nivekcode#242
Copy link
Owner

@nivekcode nivekcode left a comment

Choose a reason for hiding this comment

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

@JoA-MoS The changes look good, I guess we now have trailing comma due to the prettier bump which probably also introduced a change in the defaults. I personally don't have strong opinions on wether to use trailing comma or not. Therefore relying on the prettier default is fine by me.

Thx a lot for the PR and the awesome work here 💪

@nivekcode
Copy link
Owner

@all-contributors please add @JoA-MoS for bug code ideas test

Copy link
Contributor

@kreuzerk

I've put up a pull request to add @JoA-MoS! 🎉

@nivekcode nivekcode merged commit 606097e into nivekcode:master Feb 18, 2024
@nivekcode
Copy link
Owner

🎉 This PR is included in version 12.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

2 participants