Skip to content

Conversation

arkgil
Copy link
Contributor

@arkgil arkgil commented Jun 14, 2018

This PR fixes the bug when ExDoc included documentation for types with @typedoc false.

Closes #857

@michalmuskala
Copy link
Member

Why there are there changes to the front-end code as well?

Other than that, this looks good to me.

@josevalim
Copy link
Member

As @michalmuskala said, it looks great, you should just remove the front end changes. :)

@arkgil arkgil force-pushed the exclude-typedoc-false branch from 5c0267a to 27c0de9 Compare June 14, 2018 20:58
@arkgil
Copy link
Contributor Author

arkgil commented Jun 14, 2018

Yeah, I wasn't sure about it. I've just read the contributing guide and committed them. Anyway, I've removed these changes 🙂

@josevalim josevalim merged commit 179d4e4 into elixir-lang:master Jun 14, 2018
@josevalim
Copy link
Member

❤️ 💚 💙 💛 💜

@chevinbrown
Copy link

Thanks for the spot, @arkgil!

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

Successfully merging this pull request may close these issues.

@typedoc false not honored
4 participants