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

ubuntu-image docs fixes & updates #1930

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Conversation

rkratky
Copy link
Contributor

@rkratky rkratky commented Mar 8, 2024

Follow up to #1929. Minor fixes, updates, and additions to the ubuntu-image reference doc.

Copy link
Contributor

@upils upils left a 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:

They are not marked as resolved so I do not know if you missed them or did not deal with them en purpose.

@rkratky
Copy link
Contributor Author

rkratky commented Mar 12, 2024

LGTM but it looks like you missed 3 comments from the previous review:

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.

@rkratky
Copy link
Contributor Author

rkratky commented Mar 12, 2024

@upils I added the remaining fixes based on yyour review. Apologies for having missed those before.

Copy link
Contributor

@upils upils left a 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.

doc/reference/ubuntu-image.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@upils upils left a comment

Choose a reason for hiding this comment

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

LGTM

@rkratky rkratky force-pushed the ubuntu-image-fixes branch 3 times, most recently from e652d78 to 500c8bf Compare September 10, 2024 11:03
@rkratky rkratky merged commit f061414 into canonical:main Sep 10, 2024
3 checks passed
@rkratky rkratky deleted the ubuntu-image-fixes branch September 10, 2024 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants