-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Reopen issue 346 - Include file-content as part of markdown #1159
Comments
+1 |
2 similar comments
+1 |
+1 |
Subscribe works better than +1 !!! |
+1 |
3 similar comments
+1 |
+1 |
+1 |
I have been working (in a private repo) on a Ruby class, I've seen in (much) earlier issues that GitHub has security concerns. Should I share their concern, and not go public? |
Fwiw, I have made public my GitHub repository MarkdownHelper. The first feature is file inclusion for GitHub markdown, supporting included text as:
|
I wouldn't even mind if it were as simple as including an entire file inline. |
Then the verbatim inclusion might be helpful to you? |
+1 |
Would be awesome to include .js files in markdown documentation |
I've put up a markdown helper with file inclusion on RubyGems: |
@BurdetteLamar I am not clear what your markdown helper does exactly. What do I add to my README.md or README.rst on Github to use your markdown helper to include a file within it? |
Specify each file inclusion via an include pragma, which has the form:
where:
[The formatting of my doc is correct on GitHub, but gets mangled on RubyDoc.info. In answering you, I may have discovered a fix.! Will work on it.] |
@BurdetteLamar Do I need to add markdown_helper-0.1.5.gem in the root of my Github repo for it to work? If not, where should it go? |
@mcarans First, thanks for the earlier nudge. Got my doc fixed so it's right both on GitHub and RubyGems. Second, let's take this offline, say, to email at burdettelamar@yahoo.com. Or Skype at burdette.lamar. |
I've seen over at GH that @mcarans often(?) codes in Python. My method MarkdownHelper#include is not all that much code, so it wouldn't be hard to port it from Ruby to Python. But that idea brings me back to something I'd thought before: Maybe we should have an agreed-upon specification for include files (i.e., specify the format for the include pragma). Then implementations that honor that spec would be able to process anyone's input. Alternatively, there could be just two "universal" implementations, one (say, sh) that works on Linux, and one (say, Powershell) that works on Windows. Thoughts? [I know this is a bit off-topic for GH, but I don't know where else to go with this idea. Suggestions welcome.] |
Just in case anyone else was confused, @BurdetteLamar kindly explained to me that his markdown helper requires running before pushing to GitHub ie. it takes a template and generates the README.md from that template with any include files incorporated. You then check in the generated README.md. Combined with a Git hook, it could be a helpful automated workaround. The proper solution is for GitHub to add this functionality directly so we do not need any workarounds. |
"Proper solution" indeed. A main difference is that at GH, the inclusion could be done at render time. This would mean that the project's README.md would be the template, instead of being manufactured from the template and uploaded separately. :( |
+1 |
1 similar comment
+1 |
As I said earlier, I've put up gem markdown_helper, which does file inclusion and a couple of other things. |
+1 |
I have same docs in different languages. The reason why I need such a feature is, that the code will be equal in every language file. But if the code changes I had to change it in every language. |
+1 Would be great to be able to do this. In both the wiki and repo. Have a standard copyright, help, quick jump links back and forth, company policy information, etc... Edit one file and change it everywhere. |
+1 |
+1 |
+1 It makes it easier to keep an organization's github-pages site in sync with the main repo's markdown. Ideally, one could store the common content in a separate file and have it included by other markdown documents. Currently, If you have content which should be seen (not linked) on a README.md but also seen on a github-pages site then you must duplicate the content or include the whole README.md on the site. I don't think this is preferred for most use cases. It would be even better if we could include a particular section of a separate markdown document. What I'm advocating for is not including source files or external content but rather just other markdown from the same repository which shouldn't pose a security problem. |
+1 |
3 similar comments
+1 |
+1 |
+1 |
+1 |
2 similar comments
+1 |
+1 |
It would be an idea to use the 👍 reaction to the issue to tally support, rather than flood the issue tracker with "+1" comments, now that Github actually does support that. |
+1 |
1 similar comment
+1 |
Don't forget: you can do markdown file inclusion with Ruby gem markdown_helper. Works via both CLI and API. The documentation is over at GitHub. |
+1 |
+1 After refactoring my Wiki I need to maintain moved pages available in old locations as there are links in some forums pointing there. Without content inclusion I have to duplicate moved content in maintained old version of the page, |
+10 |
4 years - no assignee, no milestone ... a lot of people have the need and not much of a reaction yet ... |
+1 |
4 similar comments
+1 |
+1 |
+1 |
+1 |
This repository is for the I understand this is something that people want, but opening an issue here won't make that happen. People who work on the code that pertains to your query do not, in general, watch this repository. (I do because I maintain this gem.) To get it in front of their eyes, you'll need to go contact support so they can triage and prioritise it along with other requests. Just to be super clear: another "+1" comment will not make this feature request known to the support team, who are responsible for triaging feature requests. You'll need to contact them to do so. We currently have no plan to implement this feature. Thanks for your understanding. I'll be locking this issue now to prevent further pile-ons, as this is the third issue in |
Please reopen #346 - just look at the comments to see the support for this feature
The text was updated successfully, but these errors were encountered: