-
Notifications
You must be signed in to change notification settings - Fork 32
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
Move spaceros Base Image from docker to space-ros #151
Conversation
7629161
to
c2395aa
Compare
1472b85
to
65b8d3f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks pretty good except for one minor wording change in the README.
65b8d3f
to
38bfa92
Compare
Also noted the bad issue links in all my commit messages, so I edited those and force-pushed the new messages with the appropriate link to the issue in the space ros repo. |
I'm not sure why CI is stuck on this, but it should be good to go at this point. |
a008d3b
to
9123263
Compare
9123263
to
cb71377
Compare
@mkhansenbot apologies for one more update but I changed the commit messages to point to the correct issue in the docker repo (#156). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, we have functional builds from the space-ros
repo and are good to pull them out of here.
Resolves #156
We can merge this now that the spaceros build has been moved to the space-ros repo in: space-ros/space-ros#163
This removes references, CI, and documentation for the
spaceros
image, and adds a comment to the README pointing to the new location.