Skip to content

Conversation

yordis
Copy link
Contributor

@yordis yordis commented Oct 2, 2025

Signed-off-by: Yordis Prieto [email protected]

Copy link

github-actions bot commented Oct 2, 2025

@yordis
Copy link
Contributor Author

yordis commented Oct 6, 2025

@josevalim any thoughts here thus far?

@josevalim
Copy link
Member

Sorry, a bit busy with Elixir v1.19-rc and today I was out of focus. It is in my inbox and I will review it as soon as I can.

Copy link
Contributor

@leandrocp leandrocp left a comment

Choose a reason for hiding this comment

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

Hey @yordis that's amazing! Let me share some ideas that could be useful.

I was inspecting the generated Markdown files and thinking if we could apply some changes. See the differences here https://gist.github.com/leandrocp/ee4f0ba8325b410b8650ccd26b9b2351 (CompiledWithDocs.md vs CompiledWithDocs_PROPOSAL.md)

  • Use frontmatter block to describe global values/notes
  • Use the format "Summary / Functions" similar to HTML pages (so Functions become a level 2 heading)
  • Include source links
  • Reorganize metadata, for eg: add (deprecated) and doc in summary
  • Remove links to hexdocs.pm because 1) it breaks the content and I guess that would make it harder for LLMs to parse; 2) I'm not sure it should link to html pages

You can see some examples on https://shopify.dev/docs/api/liquid/basics.md and https://vercel.com/docs/rest-api/reference/sdk.md

@yordis yordis force-pushed the yordis/add-markdown-formatter branch from e1c6f8f to a6870e3 Compare October 8, 2025 19:10
@yordis yordis marked this pull request as ready for review October 8, 2025 19:12
@yordis
Copy link
Contributor Author

yordis commented Oct 8, 2025

@leandrocp thanks for the help, about the formatting, I do not have any strong opinions of the final output, I will leave it to @josevalim and you to decide on that front, I can adjust it

@yordis yordis requested a review from josevalim October 8, 2025 19:14
Signed-off-by: Yordis Prieto <[email protected]>
@yordis yordis force-pushed the yordis/add-markdown-formatter branch from a6870e3 to 5131f1a Compare October 8, 2025 19:16
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.

3 participants