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 modules doc #186

Merged
merged 1 commit into from
Jun 14, 2023
Merged

Update modules doc #186

merged 1 commit into from
Jun 14, 2023

Conversation

sogaiu
Copy link
Contributor

@sogaiu sogaiu commented Jun 3, 2023

This PR contains miscellaneous suggestions for the modules documentation.

Perhaps the most relevant change is for updating the code to the url loader example so that it no longer uses the deprecated file/popen call. The changes were based on the content of this file.

The rest of the suggestions are fairly minor concerning prose, markup, and formatting (except in a couple of cases where the content of a comment was brought in line with relevant code).

@bakpakin bakpakin merged commit ddaab6f into janet-lang:master Jun 14, 2023
@sogaiu sogaiu deleted the update-modules-doc branch June 14, 2023 22:50
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