WasmEdge / docs

https://wasmedge.org/docs/
Apache License 2.0
17 stars 57 forks source link

Add WasmEdge Style Guide #198

Closed mhmohona closed 9 months ago

mhmohona commented 10 months ago

Explanation

This PR introduces a new file, STYLE_GUIDE.md, to the WasmEdge repository. This file contains two style guides: one for documentation and one for coding.

Related issue

https://github.com/WasmEdge/docs/issues/85

What type of PR is this

/kind documentation

Proposed Changes

alabulei1 commented 10 months ago

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


Overall, the patches reviewed in this Pull Request seem to be straightforward additions and improvements to the documentation. There are no significant issues or errors identified in any of the patches. The changes include the addition of a new FAQ page, the inclusion of a style guide, hyperlink additions and revisions, and minor wording changes. The patches aim to enhance the clarity, accuracy, and usability of the documentation for users.

The most important findings include the addition of a style guide, which will help maintain consistency in coding and documentation practices. The inclusion of hyperlinks to external resources and the revision of sentences for clarity further improve the quality of the FAQ page.

However, there are a few minor suggestions for improvement. Firstly, in one of the patches, the Signed-off-by field should contain a valid email address associated with the author's GitHub account. Secondly, the commit message in another patch could be more precise and mention the specific file(s) being updated. Additionally, there is a link to the Discord server that needs to be updated to the correct URL. Lastly, in one patch, the URL format for the hyperlinks is incorrect, and it should be corrected to avoid broken links.

In summary, the changes made in this Pull Request overall enhance the documentation and provide valuable resources for users. With some minor improvements and corrections, these patches can be merged into the codebase without any major issues.

Details

Commit 6a5c4a884bae8bb4e4587aca8bbe3bdb44d1e2c3

Key changes:

Potential problems:

Commit 8d2036363f8188b6f2d46b53cb5d1754ab375db1

Key changes:

Potential problems:

Overall, the changes look good. The addition of a style guide for documentation and coding will help ensure consistency and improve the quality of the project's code and documentation.

Commit 919948ef1d383f839cbfc7c0d8ee537490a277e8

Key changes in the patch:

Potential problems:

Overall, the changes made in this patch improve the FAQ page by adding relevant hyperlinks to external resources. The revisions to the sentences also enhance the clarity of the content.

Commit a9ee642acc2db71944cc6311452bf99c0d4800c1

Key changes:

Potential problems:

Overall, the changes seem to be improvements to the FAQ page, providing more accurate information and additional resources for users.

Commit d514133d59b37a2816065bd273762b18e3758f7e

The key change in this patch is a fix to a hyperlink in the file docs/develop/getting-started/faq.md. The original hyperlink was missing a forward slash, which caused it to navigate incorrectly.

Potential problems with this patch are:

  1. The patch only fixes a single hyperlink. It would be more efficient to combine multiple small fixes into a single patch. This would reduce the overhead of creating and reviewing multiple patches.

  2. The patch includes changes to both the English and Chinese versions of the file faq.md. It would be better to separate these changes and create separate patches for each language.

Overall, the changes in this patch appear to be straightforward and should not introduce any significant problems.

Commit 7cbf126a9ea0fe3ddce20f281cafcddff4bcd7d2

Key changes:

Potential problems:

Commit 79fe907884ba4a71f575c5275f0bc5ffb16f4d3a

Key changes:

Potential problems:

Suggested improvement: The hyperlink format should be corrected to ensure valid URLs and avoid broken links.

Commit c4e3910c110325667600d9ba995930fe1e71e677

Key Changes:

Potential Problems:

Overall, this patch seems to be straightforward and doesn't introduce any new issues. It can be considered for merging into the codebase.

Commit 09bfb4bd2bd5e27045d6254b20c687c8c061eaef

Key changes:

Potential problems:

Overall, this patch seems fine and can be merged without any issues.

Commit e83f6f79efdb1d03c2e263ef8dad86ab451319b6

Key changes:

Potential problems:

alabulei1 commented 9 months ago

Hi @mhmohona

Please check out my comments. Thanks. BTW, how did you find these issues?

mhmohona commented 9 months ago

@alabulei1 so here are the source, I had these questions while writing the plugin doc. ALso i went through the discussion to find the general questions about plugin.