Skip to content

Support Windows build systems#3

Closed
tobiasweibel wants to merge 2 commits into
OriR:masterfrom
tobiasweibel:patch-1
Closed

Support Windows build systems#3
tobiasweibel wants to merge 2 commits into
OriR:masterfrom
tobiasweibel:patch-1

Conversation

@tobiasweibel
Copy link
Copy Markdown

No description provided.

Comment thread package.json
"eol": "^0.9.1",
"handlebars": "4.0.6",
"react-docgen": "2.14.0",
"react-docgen-markdown-renderer": "1.0.1"
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An update is pending. Basically PR OriR/react-docgen-markdown-renderer#6 needs to be merged first.

@OriR
Copy link
Copy Markdown
Owner

OriR commented Mar 12, 2018

As I mentioned here, I'm working on a new version and hopefully it'll handle this :)

@OriR
Copy link
Copy Markdown
Owner

OriR commented Apr 6, 2018

Your change assumes the output of the renderer is always a string.
Technically, I haven't tested this with binary formats but I don't see a reason why it shouldn't work.

If it's still an issue, I feel it's the responsibility of each renderer to make sure it's compatible with the OS.
In which case I'll add it next version of react-docgen-markdown-renderer

I'm closing this PR, but feel free to comment if you feel there's more to it :)

@OriR OriR closed this Apr 6, 2018
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.

2 participants