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

Libvirt docs touchup #1127

Merged
merged 1 commit into from
Jun 2, 2024
Merged

Libvirt docs touchup #1127

merged 1 commit into from
Jun 2, 2024

Conversation

majcosta
Copy link
Contributor

@majcosta majcosta commented Jun 1, 2024

if accepted, please squash

@majcosta majcosta requested a review from JJRcop as a code owner June 1, 2024 23:14
@majcosta
Copy link
Contributor Author

majcosta commented Jun 1, 2024

Rationale:
image

@gnif
Copy link
Owner

gnif commented Jun 1, 2024

Looks good but please squash these two commits into one and use the naming convention for commits on this project, ie:

[doc] kvmfr/libvirt: change double quotes to single quotes

Also be sure to add yourself to the AUTHORS file

@gnif gnif requested a review from JJRcop June 2, 2024 00:46
@gnif
Copy link
Owner

gnif commented Jun 2, 2024

@JJRcop until the issues I raised are addressed this still needs work.

@JJRcop
Copy link
Collaborator

JJRcop commented Jun 2, 2024

@majcosta Because you're using the GitHub website to make these changes, unless you download Git onto your computer (or a GUI client), you cannot complete the changes requested and will have to start over with a new Pull Request, keeping in mind to name your commit following our naming convention as noted.

You may want to use github.dev to start over. It's an online VS code editor made by GitHub that lets you easily edit 2 or more files in the same commit.

Otherwise, we won't make these changes for you. We recommend downloading a git client for best results especially if you plan to continue contributing to open source projects like Looking Glass in the future.

@majcosta
Copy link
Contributor Author

majcosta commented Jun 2, 2024

ok, done

virt-manager (or libvirt itself) tends to preserve outer quotation marks but explode inner double-quotes into '&quot' which is annoying.

this looks nicer when pasted into the XML editor and applied

Also used double-quotes for the QEMU <6.2 example for consistency

PS: added myself to AUTHORS since this is my first contribution
@majcosta majcosta marked this pull request as draft June 2, 2024 01:42
@majcosta majcosta marked this pull request as ready for review June 2, 2024 01:43
@gnif gnif merged commit d21bdeb into gnif:master Jun 2, 2024
16 checks passed
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.

3 participants