Skip to content

resolve conflict#1004

Merged
fujitatomoya merged 2 commits intoros2:mergify/bp/galactic/pr-987from
iuhilnehc-ynos:mergify-bp-galactic-pr-987
Sep 9, 2022
Merged

resolve conflict#1004
fujitatomoya merged 2 commits intoros2:mergify/bp/galactic/pr-987from
iuhilnehc-ynos:mergify-bp-galactic-pr-987

Conversation

@iuhilnehc-ynos
Copy link
Copy Markdown
Contributor

@iuhilnehc-ynos iuhilnehc-ynos commented Sep 9, 2022

related to #1003

Signed-off-by: Chen Lihui lihui.chen@sony.com

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
@iuhilnehc-ynos
Copy link
Copy Markdown
Contributor Author

Could you help merge it? @fujitatomoya

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
@fujitatomoya fujitatomoya merged commit eb6dfd3 into ros2:mergify/bp/galactic/pr-987 Sep 9, 2022
fujitatomoya pushed a commit that referenced this pull request Sep 9, 2022
* support wildcard matching for params file (#987)

* support regex for params file

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* add test

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* flake8

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
(cherry picked from commit 3586009)

# Conflicts:
#	rclpy/src/rclpy/node.cpp

* resolve conflict (#1004)

* resolve conflict

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* cpp headers should be put after c

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* ament_cpplint recognizes headers with extensions in angle brackets as c system headers (#1005)

excepts some headers defines in _CPP_HEADERS

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
Co-authored-by: Chen Lihui <lihui.chen@sony.com>
tonynajjar pushed a commit to pixel-robotics/rclpy that referenced this pull request Sep 10, 2022
)

* support wildcard matching for params file (ros2#987)

* support regex for params file

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* add test

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* flake8

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
(cherry picked from commit 3586009)

# Conflicts:
#	rclpy/src/rclpy/node.cpp

* resolve conflict (ros2#1004)

* resolve conflict

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* cpp headers should be put after c

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* ament_cpplint recognizes headers with extensions in angle brackets as c system headers (ros2#1005)

excepts some headers defines in _CPP_HEADERS

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
Co-authored-by: Chen Lihui <lihui.chen@sony.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants