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 formatting of list. #217

Merged
merged 1 commit into from Mar 17, 2017

Conversation

Projects
None yet
3 participants
@AustinWise
Contributor

AustinWise commented Mar 17, 2017

In the "Migrating to automatic restore" section, there is a list within an list. It does not render properly on Github or on the website.

It is like this in the source:

1. point 1
    a. subpoint a

However in HTML, it is rendered as:

<li>point1 a. subpoint a</li>

My fix is to switch the sub list to a numbered list. I think that lettered lists might not be supported by Markdown, since CommonMark does not have them yet. And GitHub Flavored Markdown is based on CommonMark, so that at least explains why it does not render nicely on GitHub.

@saldana

This comment has been minimized.

Show comment
Hide comment
@saldana

saldana Mar 17, 2017

Collaborator

Validation status: passed

File Status Preview URL Details
docs/Consume-Packages/Package-Restore.md Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Collaborator

saldana commented Mar 17, 2017

Validation status: passed

File Status Preview URL Details
docs/Consume-Packages/Package-Restore.md Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@kraigb

This comment has been minimized.

Show comment
Hide comment
@kraigb

kraigb Mar 17, 2017

Contributor

Thanks for fixing this!

Contributor

kraigb commented Mar 17, 2017

Thanks for fixing this!

@kraigb kraigb merged commit 6e7c3df into NuGet:master Mar 17, 2017

1 check passed

OpenPublishing.Build Validation status: passed
Details

@AustinWise AustinWise deleted the AustinWise:fixRestoreList branch Mar 18, 2017

@AustinWise

This comment has been minimized.

Show comment
Hide comment
@AustinWise

AustinWise Mar 18, 2017

Contributor

Thanks for the docs! I was able to migrate some old projects using those instructions.

Contributor

AustinWise commented Mar 18, 2017

Thanks for the docs! I was able to migrate some old projects using those instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment