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

Optimized for clipboard copy action. #254

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

primeagen-rustaceans
Copy link
Contributor

@primeagen-rustaceans primeagen-rustaceans commented Oct 2, 2024

Explanation

  • When marking a code snippet with bash, $ should ideally be ignored.
  • Also, the primary purpose should be to guide user How to run and output can be omitted.

Related issue

What type of PR is this

/kind documentation

Proposed Changes

  • Removed the $ in bash code snippets.
  • Removed the output of the commands.

- When marking a code snippet with bash, $ should ideally be ignored.
- Also, the primary purpose should be to guide user `How to run` and output can be omitted.

Signed-off-by: Aditya Poddar <115938518+primeagen-rustaceans@users.noreply.github.com>
Copy link
Member

juntao commented Oct 2, 2024

Hello, I am a PR summary agent on flows.network. Here are my reviews of code commits in this PR.


Commit 8a538e01347fb2da68542aa58b20ff4762561942

:
The pull request aims to optimize the documentation for better clarity and ease of understanding. Here are the key changes:

  1. Simplified code snippets by removing superfluous outputs.
  2. Enhanced guidance on how to use certain commands, making it more straightforward for users to run specified functions or programs.
  3. Optimized AoT compilation instructions with clearer step-by-step guidelines.

@alabulei1
Copy link
Collaborator

Good Catch! Thank you.

@alabulei1 alabulei1 merged commit 524e624 into WasmEdge:main Oct 7, 2024
6 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.

3 participants