-
Notifications
You must be signed in to change notification settings - Fork 105
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
add support for licenses #15
Conversation
I will keep iterating on this while processing the current repos:
|
8c2a9d7
to
706d60f
Compare
See the License for the specific language governing permissions and | ||
limitations under the License.""", | ||
|
||
"""\ |
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.
Wouldn't it be better to move the text of the license to a separate file and read it?
@esteve I moved the apache2 snippets to separate files and modified the behavior to not replace the copyright template in the LICENSE files. All repos have been updated accordingly. |
@@ -2,11 +2,18 @@ ament_copyright | |||
=============== | |||
|
|||
Checks C / C++ / CMake / Python source files for the existance of a copyright | |||
notice. | |||
and licence notice. |
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.
s/licence/license/
+1 |
The changes to the other repositories look good to me, still reviewing the code in this pr. |
3bf1508
to
7f05f31
Compare
Looks good to me. +1 |
lgtm |
Combined copyright and license checking.
Currently implemented entry points:
Connects to #10