-
Notifications
You must be signed in to change notification settings - Fork 6
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
Updating copyright headers to match current best practices #10
Conversation
d6fd964
to
357cece
Compare
LICENSE
Outdated
@@ -1,4 +1,4 @@ | |||
Copyright (C) 2019 SymbiFlow Project Authors. All rights reserved. | |||
Copyright (C) 2019 SymbiFlow Project Authors. |
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.
Expand copyright date.
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.
Fixed
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.
Please go through the checklist at f4pga/ideas#47 and make sure everything is covered then re-request review.
The files inside https://github.com/SymbiFlow/vtr-xml-utils/tree/master/tests don't have the headers yet.
|
357cece
to
b128be9
Compare
@daniellimws Added the license header to the test script |
There's also this in the checklist.
|
b128be9
to
1f4cedd
Compare
@daniellimws I've added the |
5ba664a
to
2d41858
Compare
2d41858
to
2a119db
Compare
@mithro Looks like this can be merged already |
Please rebase on master |
Signed-off-by: Maciej Kurc <[email protected]>
2a119db
to
1dc8b4f
Compare
Signed-off-by: Maciej Kurc <[email protected]>
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.
LGTM!
Signed-off-by: Maciej Kurc [email protected]