-
Notifications
You must be signed in to change notification settings - Fork 49
feat: update prettier config #23
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
Conversation
73e8317 to
ef4ff88
Compare
ef4ff88 to
adf09fe
Compare
| .get('/api/auth/me') | ||
| .set('Authorization', 'Bearer invalid-token') | ||
| .expect(401); | ||
| return request(app.getHttpServer()).get('/api/auth/me').set('Authorization', 'Bearer invalid-token').expect(401); |
Check failure
Code scanning / CodeQL
Hard-coded credentials Critical test
authorization header
adf09fe to
479ddd4
Compare
0cf90c1 to
4caeb2e
Compare
4caeb2e to
0afcdc8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not clear why you removed prettier configuration.
| "printWidth": 120, | ||
| "singleQuote": true, | ||
| "trailingComma": "all", | ||
| "printWidth": 100, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not clear why you removed prettier configuration.
Hey man, basically just because the values are the default values from prettier... the only change that is not a default value is the print width (default is 80) |
Description
This PR change prettier config and reformat the code using the new values... Merge this first #22
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
Checklist: