-
Notifications
You must be signed in to change notification settings - Fork 61
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
Remove old copyright #118
Remove old copyright #118
Conversation
@@ -1,18 +1,3 @@ | |||
/* |
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.
Add new copyright header after removing this old one?
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.
added
@@ -1,27 +1,12 @@ | |||
/* |
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.
Add new copyright header
@@ -1,18 +1,3 @@ | |||
/* |
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.
Add new copyright header and check other files?
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.
Got it will check other files, didn't realize some files didn't have the other headers
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.
@ylwu-amzn @ohltyler Should I add a copyright header to files like package.json
and .prettierrc
also? A few more config files never had at any point a copyright header
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.
If possible, sure. I remember there was some issue adding comments to some of those files, but if it compiles and works, go ahead.
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 are right, can't add comments to json files and some others
@amitgalitz regarding reformatting changes - were you able to find a VS Code extension that is equivalent to running |
I still have not been able to find why the extension is slightly off from the cli or a different extension where I can guarantee output is 100% the same. |
61e56b6
to
3f2cf6d
Compare
Reviewed each file that was changed to make sure nothing else was deleted except copyright and that each file that was changed has the new header |
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
3f2cf6d
to
bf1fd15
Compare
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
bf1fd15
to
f1d337a
Compare
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
Signed-of-by: Amit Galitzky
Description
Issues Resolved
#101
Check List
--signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.