Skip to content

Conversation

@RobDolinMS
Copy link
Collaborator

Update the Table of Contents section of the ReadMe.md to match the order of the merged MarkDown files in the printable HTML and PDF outputs

Signed-off by Rob Dolin [email protected]

Update the Table of Contents section of the ReadMe.md to match the order of the merged MarkDown files in the printable HTML and PDF outputs

Signed-off by Rob Dolin <[email protected]>
@RobDolinMS
Copy link
Collaborator Author

Tagging @vbatts so he sees this before generating the v0.4 merged printable files.

@wking
Copy link
Contributor

wking commented Mar 8, 2016

It looks like 7d55248 is missing a Signed-off-by 1, and the README
self-reference is a bit odd, but other than that, this looks good to
me.

@RobDolinMS
Copy link
Collaborator Author

Thanks @wking and @vbatts. I made the edit via GitHub on the web so signed-off by is in my commit notes which show as the first comment on this thread. Self-reference to the ReadMe is intentional as this also doubles as the Table of Contents for the merged HTML and PDF docs when we cut a version.

@mrunalp
Copy link
Contributor

mrunalp commented Mar 9, 2016

@RobDolinMS Could you amend the commit and repush?

git commit --amend --signoff

@vbatts
Copy link
Member

vbatts commented Mar 9, 2016

carried this PR to #337 and fixed the cap-sensitive file names too.

@vbatts vbatts closed this Mar 9, 2016
@RobDolinMS
Copy link
Collaborator Author

Thanks @vbatts :)

@wking wking mentioned this pull request Mar 10, 2016
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.

4 participants