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: add prettier packages #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

andresmeidla
Copy link

@andresmeidla andresmeidla commented Dec 1, 2024

Thank You for this awesome project - it really speeds things up for me.

I noticed the prettier rules were not applied: the generated project assumes that prettier has been installed in VSCode as an extension and set up correctly.
I think this might work for some people, for me - it didn't and I have no idea why to be completely honest.

I was able to get things working by adding "eslint-config-prettier", "eslint-plugin-prettier" and "prettier" itself to my aoc workspace, confed them in eslint.config.js and got it working.

In theory, the explicitly installed prettier packages should work out of the box for most people.

Made the same changes to the generator, but I have no way of validating whether this works, though - I wasnt able to use the locally built generator, it'll always use the one published in npm 🤷

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.

1 participant