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

Add Github actions for colcon build (backport #430) #439

Merged
merged 6 commits into from
Feb 17, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Feb 17, 2024

This is an automatic backport of pull request #430 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
(cherry picked from commit 4ff0841)
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
(cherry picked from commit 1ba828b)
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
(cherry picked from commit a354a7e)
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
(cherry picked from commit 9fbac16)
* Current maintainers don't have access to fix CircleCI

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
(cherry picked from commit ab02f52)
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@Ryanf55
Copy link
Collaborator

Ryanf55 commented Feb 17, 2024

FYI @wep21 - I took the backport and then added 1e2b9cd on top to change the environment from rolling to iron.

Because iron and humble have nav2 binaries available, it might make sense to just use those instead of compiling from source as an optimization to speed up CI.

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Feb 17, 2024

Merging this as-is because it passes CI. I can optimize for speed later.

@Ryanf55 Ryanf55 merged commit d8727ce into iron Feb 17, 2024
2 checks passed
@Ryanf55 Ryanf55 deleted the mergify/bp/iron/pr-430 branch February 17, 2024 17:22
@Ryanf55 Ryanf55 added ros2 Affects ROS 2 ci Continuous Integration labels Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration ros2 Affects ROS 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant