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

Update documentation for Juvix 0.6.8 #124

Merged
merged 29 commits into from
Nov 12, 2024
Merged

Update documentation for Juvix 0.6.8 #124

merged 29 commits into from
Nov 12, 2024

Conversation

lukaszcz
Copy link
Contributor

@lukaszcz lukaszcz commented Nov 4, 2024

Major rewrite of the documentation

@lukaszcz lukaszcz self-assigned this Nov 4, 2024
@jonaprieto
Copy link
Collaborator

jonaprieto commented Nov 4, 2024

The preview of this PR failed to build. Please check the logs.

docs/Package.juvix Outdated Show resolved Hide resolved
@@ -131,11 +131,6 @@ plugins:

nav:
- ./README.juvix.md
- About:
Copy link
Collaborator

@jonaprieto jonaprieto Nov 8, 2024

Choose a reason for hiding this comment

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

I read the Changelog from time to time. Why do you want to remove it?
I'd go even further and promote it as its own tab.
The rest of the info in About may be useful for some people; I'd suggest moving to the end, perhaps after Discord.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The "About" section mostly duplicated information available elsewhere, e.g., the front page, sometimes even repeating the same sentences. We want to keep information relevant for Juvix newcomers, save them from information overload and allow them to find relevant info about using Juvix quickly. If someone wants to see the Changelog, they can go to github. As the Changelong is not relevant to actually using Juvix, I don't see a reason to keep it in the language documentation.

But the "Talks" subpage should be preserved

Comment on lines 19 to -48

## Related internal reports

An ongoing effort to specify the Juvix language and internal representations
along its semantics is being carried out in the form of internal reports. These
reports are written for our own internal use and are not intended to be read by
the public, at least for now. However, we are making them available here
for the sake of transparency.

<div style="text-align:center" markdown>

| Title | Doi/url | Date |
| :------------------------- | :---------------------------------------------------------- | :----------- |
| Juvix to VampIR Pipeline | [10.5281/zenodo.8268823](https://zenodo.org/record/8268823) | Aug 14, 2023 |
| The Core language of Juvix | [10.5281/zenodo.8297159](https://zenodo.org/record/8297159) | Aug 29, 2023 |
| Geb Pipeline | [10.5281/zenodo.8262747](https://zenodo.org/record/8262747) | Aug 21, 2023 |
| Rethinking VampIR | [10.5281/zenodo.8262815](https://zenodo.org/record/8262815) | Aug 29, 2023 |

</div>
Copy link
Collaborator

@jonaprieto jonaprieto Nov 8, 2024

Choose a reason for hiding this comment

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

I'd keep this information (perhaps rewrite/simplify the paragraph, change the title) and update the table. Also, add @lukaszcz's latest report on Cairo. I'm sure some people find these reports useful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added back the reports, though they don't really fit into "Language reference" -- they are about compiler implementation, not the language itself.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe I move this to separate subpage under "Reference".

@jonaprieto
Copy link
Collaborator

jonaprieto commented Nov 8, 2024

Success! The preview of this PR will be available at https://docs.juvix.org/pull-124/ in a few minutes.

Please note that this link will be deleted when the PR is closed or merged.

@lukaszcz lukaszcz changed the title Update documentation for Juvix 0.6.7 Update documentation for Juvix 0.6.8 Nov 11, 2024
@lukaszcz lukaszcz marked this pull request as ready for review November 12, 2024 10:55
@paulcadman paulcadman merged commit 7471115 into main Nov 12, 2024
1 of 2 checks passed
@paulcadman paulcadman deleted the update-docs branch November 12, 2024 13:02
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.

3 participants