Fixed index.md handling

Ellerbach / docfx-companion-tools

A suite of tools, pipelines templates to take the very best of DocFX

MIT License

66 stars 18 forks source link

Fixed index.md handling #2

Closed mtirionMSFT closed 3 years ago

mtirionMSFT commented 3 years ago

INDEX.md files were ignored in handling. This is removed from the code to have proper handling.
folders not containing sub-folders were parsed differently from folders with sub-folders. Corrected this behavior.

mtirionMSFT commented 2 years ago

@sacrejohn I removed it because in both cases WalkDirectoryTree is needed. But I overlooked the different parameters. Will bring that back. Thanks for your comment :)

mtirion commented 2 years ago

@sacrejohn issue fixed. WalkDirectoryTree is now done in both cases, but href determined as before.

sacrejohn commented 2 years ago

I have looked at and tested the new version. It seems to me that the WalkDirectoryTree method should only be called in the else clause, otherwise 1 child element is created. Exactly like the inital code.