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

docs: cleanup documents #1111

Merged
merged 6 commits into from
Dec 17, 2024
Merged

docs: cleanup documents #1111

merged 6 commits into from
Dec 17, 2024

Conversation

osrm
Copy link
Contributor

@osrm osrm commented Dec 17, 2024

Summery

Fix typo and invalid links

Description

I'v been checked every ‘md’ files in the repository and found some invalid links and typo.

I fixed most of the invalid links, but #1110 (comment) thought it was better to open an issue rather than fix them arbitrarily.

I also found a minor typo and fixed it, otherwise the documentation looks perfect. Hope this helps.

osrm added 6 commits December 18, 2024 00:40
 [air.rs](./air.rs) is invalid. Changed to  [air.rs](./src/air.rs)
"[Writing the Guest Program](../../benchmarks/README.md#writing-the-guest-program" is invalid.

Changed to "[Writing the Guest Program](../../../benchmarks/README.md#writing-the-guest-program"
"[fibonacci example](./bin/fibonacci.rs)" is not working.

Changed to "[fibonacci example](./src/bin/fibonacci.rs)".
"[Overview of Basic Usage](./overview.md)" is invalid.

Changed to "[Overview of Basic Usage](../writing-apps/overview.md)".
"[github workflows](./.github/workflows/benchmark-call.yml)" is invalid link.

Changed to "[github workflows](../../.github/workflows/benchmark-call.yml)".
- "an memory access" -> "a memory access"

The phrase "an memory access" is considered incorrect because the article "an" is used before words that begin with a vowel sound. Since "memory" starts with a consonant sound, the correct article to use is "a," making it "a memory access."
Copy link
Contributor

@jonathanpwang jonathanpwang left a comment

Choose a reason for hiding this comment

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

Thanks!
CI is failing due to a permission issue which should have been fixed in #1112

@jonathanpwang jonathanpwang merged commit 3a83db0 into openvm-org:main Dec 17, 2024
7 of 11 checks passed
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