-
Notifications
You must be signed in to change notification settings - Fork 155
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
ubuntu-image docs fixes & updates #1930
Conversation
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.
LGTM but it looks like you missed 3 comments from the previous review:
- Add ubuntu-image reference #1929 (comment)
- Add ubuntu-image reference #1929 (comment)
- Add ubuntu-image reference #1929 (comment)
They are not marked as resolved so I do not know if you missed them or did not deal with them en purpose.
Not on purpose -- I didn't see them. I pbbly opened the review before you were finished, and I didn't reload. My bad. I'll add fixes for these, too. |
b3948f6
to
a10002e
Compare
@upils I added the remaining fixes based on yyour review. Apologies for having missed those before. |
a10002e
to
bb4d7c6
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.
LGTM aside from the small bit about the classic image build explanation.
bb4d7c6
to
6d7d177
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.
LGTM
e652d78
to
500c8bf
Compare
500c8bf
to
b94abac
Compare
Follow up to #1929. Minor fixes, updates, and additions to the
ubuntu-image
reference doc.