Skip to content

Touch ups to LSP and Tools packages #830

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

Merged
merged 3 commits into from
Nov 3, 2023
Merged

Touch ups to LSP and Tools packages #830

merged 3 commits into from
Nov 3, 2023

Conversation

zth
Copy link
Collaborator

@zth zth commented Oct 12, 2023

  • Changes to a more uniform naming of distributed binaries
  • Adds an example of how we could bind to the structure the docgen emits zero cost using ReScript v11

These things are fine to do because neither of the packages have had their initial versions published yet. We'll hold off publishing the initial tools version at least until we've figured out these last details.

cc @aspeddro

Also, @aspeddro did you have a basic generator from this format to markdown in some default format? Maybe something you've done when experimenting on the website?

@aspeddro
Copy link
Contributor

aspeddro commented Oct 13, 2023

There was a basic generator for markdown, but it wasn't correct.

I am exploring generating the documents through getStaticProps.

Here is the latest deploy: https://rescript-lang-fn6212fyj-rescript-association.vercel.app/docs/manual/next/api/js

PR: rescript-lang/rescript-lang.org#703

It is on the next route. The paths are the same as the module path. So the module Belt.SortArray.Int -> belt/sortarray/int

* update ci, readme and contributing.md

* fix

* add Cli.resi and rescript.json

* fixes

* restore some changes
@zth zth merged commit 4890db4 into master Nov 3, 2023
@zth zth deleted the touch-ups branch November 3, 2023 08:25
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