Skip to content

Conversation

@ChrSteinert
Copy link
Contributor

@ChrSteinert ChrSteinert commented Jun 10, 2024

This adds the section about Quadlet to the guide version of the Podman page.

Would you like this section to be further expanded for the guide version?

Author checklist (Completed by original Author)

  • Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • If applicable, steps and instructions have been tested to work
  • Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

# Introduction

!!! note
!!! note "Note"
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @ChrSteinert, I'm commenting inline here. There's no need to add the "Note" in quotes as you have done. It is an option for translators to add that in their language if they want within the Crowdin interface, however most have chosen not to do this, as in general, the admonition titles, even in English, are pretty self-explanatory. I'd prefer that you not add these within the English source documents.


!!! note
!!! note "Note"

Copy link
Contributor

Choose a reason for hiding this comment

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

See mention above on this

@sspencerwire
Copy link
Contributor

@ChrSteinert If there's more to be said within the guide that will be helpful, please feel free to add it. Thank you!

- Correct that `-d` is detach and not `-p`
- Add a tip suggesting using `--rm`
@ChrSteinert ChrSteinert marked this pull request as draft June 10, 2024 19:18
@ChrSteinert
Copy link
Contributor Author

Let me have a go through it for a few days maybe! :)

@ChrSteinert
Copy link
Contributor Author

That's what I have for now.

@github-actions
Copy link

Test results for cb75404:

Number of broken URLs: 1

URL,RESULT,FILENAME
 https://azuremarketplace.microsoft.com/en-us/marketplace/apps/resf.rockylinux-aarch64,failed,guides/cloud/migration-to-new-azure-images.md

@sspencerwire
Copy link
Contributor

That's what I have for now.

If you are ready for a review and merge @ChrSteinert , just take it out of draft mode.

@ChrSteinert ChrSteinert marked this pull request as ready for review June 13, 2024 12:57
@ChrSteinert
Copy link
Contributor Author

Stupid me – thanks for the heads up!

Copy link
Contributor

@sspencerwire sspencerwire left a comment

Choose a reason for hiding this comment

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

Thanks @ChrSteinert .. light editing to follow

@sspencerwire sspencerwire merged commit 1f1c45d into rocky-linux:main Jun 17, 2024
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