Skip to content
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

Remove file include syntax from Markdown #143

Merged
merged 1 commit into from
Oct 30, 2019

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Oct 30, 2019

We decided to go with pure Markdown for now for simplicity reason and not allow users to include other files. Because of this I am removing these examples.

We decided to go with pure Markdown for now for simplicity reason and not allow users to include other files. Because of this I am removing these examples.
@ruflin
Copy link
Contributor Author

ruflin commented Oct 30, 2019

@exekias This change means, it is up to the builder of the package to make sure all docs are included in one README file together.

@hbharding You brought up yesterday the point around what we could specify in the manifest file to have it generated. I think one of these things are the input docs. My thinking is that each input should be able to have its own documentation file. If there is one, we should include it automatically. (in the future).

@ruflin ruflin requested a review from exekias October 30, 2019 07:57
@ruflin ruflin merged commit 9efb66b into elastic:master Oct 30, 2019
@ruflin ruflin deleted the remove-include-syntax branch October 30, 2019 10:08
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