Skip to content
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

Feat review package ros #2

Merged
merged 6 commits into from
Dec 11, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# CODEOWNERS info: https://help.github.com/en/articles/about-code-owners
# Owners are automatically requested for review for PRs that changes code
* @ronaldoevangelista
39 changes: 39 additions & 0 deletions .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
---
name: Bug report
about: Create a report to help us improve
title: ''
labels: ''
assignees: ''

---

### Configuration

- Operating system: \<Ubuntu 00.00.0 LTS\>

- Package version: \<0.0.0\>

### Description of problem

- \<Currently it is possible to build ...\>

### Steps to Reproduce

1. step
2. step

### Actual Results

- \<API internal error\>

### Expected Results

- \<Successful build\>

### Related Package/PR

- N/A.

### Notes

- N/A.
22 changes: 22 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@

#### **Overview**

- \<Overview for request or description for added feature.\>
- \<Briefly describe the feature, a short description of the requested feature.
Images can be included to improve the understanding of the functionality.\>

#### **What was added/changed in this update**

- \<Use the commit messages to describe what was included.\>

#### **Depends On:**

- \<Inform if there is a dependency on other pull requests.\>

#### **Related Issues:**

- \<Inform if the pull request is associated with issues.\>

#### **Notes:**

- Include notes and notes specific to this pull request here, if not, include the expression N/A. (Not Applicable)
52 changes: 52 additions & 0 deletions .github/workflows/colcon-action-runner.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
# Workflow: colcon-action-runner
#
# This GitHub Actions workflow, named "colcon-action-runner,"
# is designed to be triggered on workflow calls.
# This action build, tests and run coverage with ROS or ROS 2 workspace from source.
#
# Trigger
# This workflow is triggered by a `workflow_call` event.
# Thirdparty actions
# https://github.com/marketplace/actions/ros-2-ci-action
# Comments a pull request with the code coverage generated by your tests.
# https://github.com/marketplace/ros-tooling/setup-ros

name: colcon-action-runner

on: [pull_request, workflow_dispatch]

jobs:
setup-colcon-ci:
strategy:
fail-fast: false
matrix:
ubuntu: [jammy]
distro: [humble]
runs-on: ubuntu-latest
container:
image: rostooling/setup-ros-docker:ubuntu-${{ matrix.ubuntu }}-latest

name: run ${{ matrix.distro }} on colcon
steps:
- name: setup ros environment
uses: ros-tooling/setup-ros@v0.7
with:
required-ros-distributions: ${{ matrix.distro }}
- name: build - ${{ github.event.repository.name }}
uses: ros-tooling/action-ros-ci@v0.3
with:
package-name: |
${{ github.event.repository.name }}
target-ros2-distro: ${{ matrix.distro }}
skip-tests: true
no-symlink-install: true
colcon-extra-args: --event-handlers console_direct+ --executor sequential
colcon-defaults: |
{
"build": {
"cmake-args": [
"-DCMAKE_BUILD_TYPE=Debug"
]
}
}
colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/1ddb69bedfd1f04c2f000e95452f7c24a4d6176b/index.yaml
43 changes: 43 additions & 0 deletions .github/workflows/runner-ci-lint.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
# This file configures a set of analyzers, which statically check code and configuration files
# using the following tools copyright, cppcheck, cpplint, gtest, lint_cmake, linter, uncrustify,
# xmllint. These tools look for formation errors, patterns and other warnings.
#
# Checkout This action checks-out your repository under $GITHUB_WORKSPACE,
# so your workflow can access it.
# https://github.com/actions/checkout
# https://github.com/marketplace/actions/checkout
#
# ROS 2 Tooling Working Group
# Github action to run linters on ROS 2 packages
# https://github.com/ros-tooling/action-ros-lint
# rostooling/setup-ros-docker:ubuntu-jammy-ros-humble-desktop-latest
# https://github.com/ros-tooling/setup-ros-docker
# https://hub.docker.com/r/rostooling/setup-ros-docker/tags

name: lint-runner

on: [pull_request, workflow_dispatch]


jobs:
ament_lint_general:
name: ament_${{ matrix.linter }}
runs-on: ubuntu-latest
timeout-minutes: 20
container:
image: rostooling/setup-ros-docker:ubuntu-${{ matrix.ubuntu }}-latest
strategy:
fail-fast: false
matrix:
linter: [copyright, lint_cmake, xmllint]
distro: [humble]
ubuntu: [jammy]
package: [sonar_msgs]
steps:
- uses: actions/checkout@v4
- uses: ros-tooling/setup-ros@0.7.8
- uses: ros-tooling/action-ros-lint@master
with:
linter: ${{ matrix.linter }}
distribution: ${{ matrix.distro }}
package-name: ${{ matrix.package }}
9 changes: 9 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# Changelog

## [Unreleased]

### Added

- Added First version

[Unreleased]: https://github.com/Brazilian-Institute-of-Robotics/sonar_msgs
44 changes: 20 additions & 24 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,31 +1,27 @@
cmake_minimum_required(VERSION 2.8.3)
cmake_minimum_required(VERSION 3.8)

project(sonar_msgs)

## Compile as C++11, supported in ROS Kinetic and newer
# add_compile_options(-std=c++11)
set(ignoreMe "${BUILD_DOCS}")

## Find catkin macros and libraries
## if COMPONENTS list like find_package(catkin REQUIRED COMPONENTS xyz)
## is used, also find other catkin packages
find_package(catkin REQUIRED COMPONENTS
std_msgs
message_generation
)
find_package(ament_cmake REQUIRED)
find_package(std_msgs REQUIRED)
find_package(rosidl_default_generators REQUIRED)

## System dependencies are found with CMake's conventions
# find_package(Boost REQUIRED COMPONENTS system)
set(msg_files
"msg/Sonar.msg"
"msg/SonarStamped.msg"
)

add_message_files(
FILES
Sonar.msg
SonarStamped.msg
)
rosidl_generate_interfaces(${PROJECT_NAME}
${msg_files}
DEPENDENCIES std_msgs
)

generate_messages(
DEPENDENCIES
std_msgs
)
if(BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
set(ament_cmake_cpplint_FOUND TRUE)
ament_lint_auto_find_test_dependencies()
endif()

catkin_package(
CATKIN_DEPENDS message_runtime
)
ament_package()
13 changes: 13 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
Any contribution that you make to this repository will
be under the Apache 2 License, as dictated by that
[license](http://www.apache.org/licenses/LICENSE-2.0.html):

~~~
5. Submission of Contributions. Unless You explicitly state otherwise,
any Contribution intentionally submitted for inclusion in the Work
by You to the Licensor shall be under the terms and conditions of
this License, without any additional terms or conditions.
Notwithstanding the above, nothing herein shall supersede or modify
the terms of any separate license agreement you may have executed
with Licensor regarding such Contributions.
~~~
Loading
Loading