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

Update ros-lint #1786

Merged
merged 3 commits into from
Aug 20, 2024
Merged

Update ros-lint #1786

merged 3 commits into from
Aug 20, 2024

Conversation

mosfet80
Copy link
Contributor

Upgrade to Node.js v20

Upgrade to Node.js v20

Signed-off-by: mosfet80 <[email protected]>
@mosfet80 mosfet80 requested a review from a team as a code owner August 10, 2024 07:31
@mosfet80 mosfet80 requested review from MichaelOrlov and hidmic and removed request for a team August 10, 2024 07:31
Signed-off-by: mosfet80 <[email protected]>
Signed-off-by: mosfet80 <[email protected]>
Copy link
Contributor

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, github workflow is also green.

Copy link
Contributor

@MichaelOrlov MichaelOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MichaelOrlov MichaelOrlov merged commit 270884a into ros2:rolling Aug 20, 2024
12 checks passed
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.

3 participants