-
Notifications
You must be signed in to change notification settings - Fork 71
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
Prettier setting in config file in the repo should apply all contributors code automatically #94
Comments
If we use a pre-commit hook to format with every commit, will that work? As prettier will already we installed as a dev dependency. |
i am not familiar with pre-commit hooks, if that solves the problem it's great actually. could you take care of that? |
I've also never implemented this myself but I want to pick this up. Could you please edit the Issue title and assign this ticket to me? |
edited and assigned, thank you. |
Facing hurdles on this as we have mono repo kind of setup and preCommit is struggling as there are mutiple package.json and .git file is not in same folder as in package.json. I'm still trying will keep updated. |
thank you for the update. |
@AbhiBajpaishek, are you still working on this? I know this is a tricky issue. I also couldn't figure it out. I will remove an assignment from this to see if it can catch the attention of someone with a solution. Please still let me know if you find out. |
@lugenx Hi I have some idea on how to use pre-commit hooks to resolve this issue. Can I work on it? |
hi @AyanDas348 , sure, please go ahead. I have assigned this issue to you 👍 |
hi @lugenx , if this issue is open, assign it to me, please. Thanks |
sure, i assigned this to you @itegika, please make sure that you read all the comments above before starting to work. let me know if you have any questions. |
Sure, thank you) |
Hi, @lugenx , sent PR regarding this issue https://github.com/lugenx/ecohabit/pull/233 |
Hi, @lugenx found a solution using husky. Is that an option available? |
No description provided.
The text was updated successfully, but these errors were encountered: