-
Notifications
You must be signed in to change notification settings - Fork 185
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
Enforce Prettier on commits and builds. #36
Comments
I am a participant of SWoC and I will like to work on this issue. I use both VS Code and Prettier extension and I am familiar with it. Please assign this issue to me so that I can start working on it. |
@praveenscience sir, can you explain a bit how can we do the same? |
@Sloth-Panda Something like this: https://gist.github.com/jareware/7179093 |
@Sloth-Panda and @sonaliBedade Check these out:
I guess these might be a good start... |
I would like to work on this issue, @praveenscience kindly assign it to me! |
Hey @dollyjain51099, sure... You can also discuss with @sonaliBedade, as you both know each other! 😅 |
Hey !! I 'm a participant of GSSOC'21, can you assign this issue to me? |
@sahildotexe It's already being done by @dollyjain51099 and @sonaliBedade... Check if there are issues that are not claimed. Also check with them, they're struggling on bringing this to CI CD. |
@praveenscience should this issue be closed as you mentioned that we are planning to use something like husky. |
@ampsteric This is totally different. This is for the CI CD Pipeline. |
Ohh okay , actually I saw your comment #41 (review) and thought of asking. |
I have got this prettier options configured in VS Code. We need a way to ensure that this is there in the build times. If it's not used, including the spaces, etc., the build should fail.
This issue is up for grabs for both DWoC and SWoC.
The text was updated successfully, but these errors were encountered: