-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Formattted everything with prettier #1879
Changes from 1 commit
cc4e9aa
64b2f2b
540d852
d575e5a
7a1192e
4510bff
2c235f6
3bfb27f
b4888be
eddbe5e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -47,7 +47,6 @@ gulp.task('tslint', function() { | |
return merge(srcs, tests); | ||
}); | ||
|
||
|
||
gulp.task( | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why not use https://www.npmjs.com/package/gulp-prettier? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Mainly because that package isn't maintained, and it's missing the "es5" trailing commas option. |
||
'prettier', | ||
shell.task([ | ||
|
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.
You could have called the
gulp
task here?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.
I was thinking that we should be checking all the files to see if any of them haven't been prettified.
I think I set up
gulp prettier
to only prettify files that have changed since the last commit.Come to think of it, I think this workflow can be improved...