-
Notifications
You must be signed in to change notification settings - Fork 69
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
Quality declarations for rmw and rmw_implementation_cmake #205
Conversation
d80ff77
to
0e75fac
Compare
Updating to reflect current QL status (4). |
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.
some comments and links
|
||
### License [3.iii] | ||
|
||
The license for `rmw_implementation_cmake` is Apache 2.0, and a summary is in each source file, the type is declared in the `package.xml` manifest file, and a full copy of the license is in the `LICENSE` file. |
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.
The license for `rmw_implementation_cmake` is Apache 2.0, and a summary is in each source file, the type is declared in the `package.xml` manifest file, and a full copy of the license is in the `LICENSE` file. | |
The license for `rmw_implementation_cmake` is Apache 2.0, and a summary is in each source file, the type is declared in the [package.xml](./package.xml) manifest file, and a full copy of the license is in the [LICENSE](../LICENSE) file. |
Signed-off-by: Stephen Brawner <brawner@gmail.com>
0e75fac
to
5247b6e
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.
LGTM - fixing the comments
Including aspirational quality declarations for
rmw
andrmw_implementation_cmake
. While this only has one commit, it does technically depend on #203 and #204 and should be rebased when they are merged.Signed-off-by: Stephen Brawner brawner@gmail.com