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

Fix appliance backup and restore numbered lists #1758

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Oct 27, 2023

a/b/c are not valid Markdown. Also, need at least 3 spaces for nested items in numbered lists.

@rwellon @bdunne Please review.

Before:
Preview_backup_and_restore_md_—_manageiq-documentation

After:
Preview_backup_and_restore_md_—_manageiq-documentation

a/b/c are not valid Markdown. Also, need at least 3 spaces for nested
items in numbered lists.
@miq-bot
Copy link
Member

miq-bot commented Oct 31, 2023

Checked commit Fryguy@821c837 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@bdunne bdunne merged commit f705481 into ManageIQ:master Nov 7, 2023
2 checks passed
github-actions bot pushed a commit to ManageIQ/manageiq.github.io that referenced this pull request Nov 7, 2023
@Fryguy Fryguy deleted the fix_backup_restore branch November 7, 2023 15:17
@Fryguy Fryguy assigned bdunne and unassigned rwellon Nov 7, 2023
@Fryguy
Copy link
Member Author

Fryguy commented Nov 8, 2023

Backported to quinteros in commit a14f5ce.

commit a14f5cefedd1d7dc3c08a3f601ed530330ed296d
Author: Brandon Dunne <[email protected]>
Date:   Tue Nov 7 09:49:46 2023 -0500

    Merge pull request #1758 from Fryguy/fix_backup_restore
    
    Fix appliance backup and restore numbered lists
    
    (cherry picked from commit f705481633e7c0a9a89f70327d119f91084c7f88)

Fryguy pushed a commit that referenced this pull request Nov 8, 2023
Fix appliance backup and restore numbered lists

(cherry picked from commit f705481)
github-actions bot pushed a commit to ManageIQ/manageiq.github.io that referenced this pull request Nov 8, 2023
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.

4 participants