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

Try out container matrix #74

Closed
wants to merge 1 commit into from
Closed

Try out container matrix #74

wants to merge 1 commit into from

Conversation

tfoote
Copy link
Member

@tfoote tfoote commented Mar 25, 2024

No description provided.

@tfoote tfoote requested a review from audrow as a code owner March 25, 2024 17:57
@rkent
Copy link
Contributor

rkent commented Mar 25, 2024

Cool, I didn't know container: ['ros:iron', 'ros:rolling'] was an option

@tfoote
Copy link
Member Author

tfoote commented Mar 25, 2024

I wasn't sure either. You can do arbitrary keys. That's why I tried it out in the PR before suggesting it in #72 I think that the setup/python might not be necessary with this new base image too.

@tfoote
Copy link
Member Author

tfoote commented Mar 25, 2024

As this was a test, I'll close this and we can integrate the work into the iteration in #72

@tfoote tfoote closed this Mar 25, 2024
@tfoote tfoote deleted the tfoote-test-containers branch March 25, 2024 18:08
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