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

.html -> .md #637

Closed
wants to merge 1 commit into from
Closed

.html -> .md #637

wants to merge 1 commit into from

Conversation

robrich
Copy link

@robrich robrich commented Dec 31, 2019

On the website it's correctly linking to install.html. On GitHub's markdown rendering, it yields a 404.

On the website it's correctly linking to install.html.  On GitHub's markdown rendering, it yields a 404.
@jsf-clabot
Copy link

jsf-clabot commented Dec 31, 2019

CLA assistant check
All committers have signed the CLA.

@icambron
Copy link
Member

icambron commented Dec 31, 2019

Won't this break the website, though? In general, the markdown doesn't "work"; it only works when compiled. For example, the links into the API docs don't work either.

I tried to capture that here: https://github.com/moment/luxon/blob/master/docs/readme.md

I'd of course love to fix it so that it works in both places.

@robrich
Copy link
Author

robrich commented Dec 31, 2019

Good point. Google landed me to the github page rather than the docs site suggesting we may need to adjust SEO details.

And yes, making this change here will break the "compiled" versions. Perhaps the compilation step could rewrite .md to .html on the way through? How do I run the compilation locally to understand the feasability of this?

@robrich
Copy link
Author

robrich commented Dec 31, 2019

Also, the build fail is really interesting. If running from Dec 31 and adding 1 day, the relative date says "next year" not "tomorrow". Is this issue-worthy?

@icambron
Copy link
Member

npm run docs. In theory you can do it with an esdoc plugin. Last time I tried to do that, I ran into trouble, but I don't recall why

@icambron
Copy link
Member

The issue about "next year" is #394

@robrich
Copy link
Author

robrich commented Dec 31, 2019

Not finding the feature in esdoc docs, I asked esdoc/esdoc#558. If this doesn't yield a built-in feature, we could pre-regex the files before building, but that seems quite heavy-handed. (Their github repo is .html files: https://github.com/esdoc/esdoc-site/blob/master/docs/manual/config.html)

@icambron
Copy link
Member

icambron commented Jan 3, 2020

In my experience, you are unlikely to get a response from the esdoc maintainer.

@icambron
Copy link
Member

I wonder if for 2.0, we should scrap esdoc and use typedoc or similar. Then the conversation about how to deploy the markdown gets reset anyway

@GillesDebunne
Copy link
Collaborator

GillesDebunne commented Jan 29, 2020 via email

@icambron
Copy link
Member

Going to close this one for now. Let's revisit when we do 2.0.

@icambron icambron closed this Mar 25, 2020
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.

4 participants