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 doc for releasing v0.13.2 #93

Merged
merged 9 commits into from
Nov 15, 2023
Merged

Conversation

apepkuss
Copy link
Collaborator

No description provided.

Copy link
Member

juntao commented Nov 15, 2023

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


Overall Summary:
This GitHub Pull Request titled "Update doc for releasing v0.13.2" includes multiple commits with various changes, mainly focusing on updating workflows, versioning tables, and documentation. While most of the changes seem to be straightforward and do not pose potential problems, there are a few issues that need attention.

The first potential problem lies in the paths-ignore configuration of the rust-static-lib workflow. It is not clear why these specific paths are added and how they may impact the desired behavior of the workflow.

The matrix configuration for the build_ubuntu job in the rust-static-lib workflow has been updated without stating the reasons for including specific Rust versions. Some context would be helpful to understand the motivation behind these changes.

Another notable issue is a typo in the cmake command in the run script, which includes the redundancy of "cmake cmake" instead of just "cmake."

Furthermore, the flag -DWASMEDGE_BUILD_TOOLS=OFF has been set in the cmake command, but it is unclear if this aligns with the intended changes, as it would exclude building the tools. Verification is needed to ensure the alignment.

Regarding the updates to the ci(rust-static-lib) workflow, the commented out Build WasmEdge with Release mode step requires clarification on whether it should be re-enabled or not. Additionally, the relationship between the Install dependencies step and the commented step needs clarification.

Overall, this pull request introduces various changes, with the most notable issues being the potential problems mentioned above. It is recommended that the author addresses these concerns and provides additional context where needed for better clarity and understanding.

Details

Commit 14614a0d44b9329af7cf7c6c275ffa01a195b6fd

Key changes:

  • Updated workflows for CI build, rust-static-lib, and standalone.
  • Added paths to ignore for pull requests in the CI build and standalone workflows.
  • Updated the matrix of operating systems and Rust versions for the build_ubuntu job in the rust-static-lib workflow.

Potential problems:

  • It's unclear why the paths .github/workflows/rust-static-lib.yml, bindings/rust/, include/api/wasmedge/, and lib/api/** are added to the paths-ignore configuration in the push and pull_request events of the rust-static-lib workflow. This may affect the desired behavior of the workflow and should be reviewed.
  • The matrix configuration in the build_ubuntu job of the rust-static-lib workflow was updated to include Rust versions 1.73, 1.72, and 1.71, but the reasons for these specific versions are not stated. It would be helpful to have some context for these changes.

Commit 214c2e2d1f3f68bc98ff20c6b5890661b2153fbd

Key changes:

  • The versioning table in the wasmedge-sys crate's lib.rs file has been updated.

Potential problems:

  • There are no potential problems with this change as it only updates a versioning table in the code documentation.

Commit fb8da6d07a056b66af02fa9440bd043ad9358914

Key changes:

  • Updated the versioning table in both the README.md and src/lib.rs files.
  • Added the version 0.13.2 to the table.

Potential problems:

  • There are no potential problems identified in this patch.

Commit 3918ea63df7ef9bd58688f618fa72972f875cea0

Key changes:

  • Fixed a typo in the .github/workflows/rust-static-lib.yml file.

Potential problems:

  • There is a typo in the cmake command in the run script. The command "cmake cmake" should be changed to "cmake" to avoid this redundancy.

Commit 28086792e812707272ec9d34adc5b41e8f967241

Key changes:

  • The job in the .github/workflows/rust-static-lib.yml file has been updated.
  • The cmake command now includes the flag -DWASMEDGE_BUILD_TOOLS=OFF.

Potential problems:

  • It seems that the WASMEDGE_BUILD_TOOLS flag has been set to OFF in the cmake command. This means that the build process will not include building the tools. This may be intentional, but it should be verified if it aligns with the intended changes in this pull request.

Commit 513f18e49f6b37ee695b0e17afd94abd23e0706c

Key changes:

  • The version number in the documentation for wasmedge-sys has been updated from 0.13.1 to 0.13.2.

Potential problems:

  • None identified.

Commit 2adbc0af890a818b06a2d3b30081690660e5c468

Key changes:

  • Bumped the version of the Rust SDK to 0.13.2 in the Cargo.toml file.

Potential problems:

  • No potential problems found.

Commit 424706faef76f8863ab2f205048a913328ac7559

Key changes:

  • Updated version numbers in the versioning table in both the README.md file and lib.rs file.
  • The updated version is 0.13.2.

Potential problems:

  • No potential problems are identified in this patch.

Commit 0cfabfe292964ca834d8bf2f974e30da543d8ac1

Key changes:

  • Updated the ci(rust-static-lib) workflow in the .github/workflows/rust-static-lib.yml file.
  • Added an Install dependencies step to install necessary software dependencies.
  • Commented out the Build WasmEdge with Release mode step.
  • Added an Install Rust-nightly step using the dtolnay/rust-toolchain@nightly action.

Potential problems:

  • The Build WasmEdge with Release mode step has been commented out. It is unclear if this is intentional or if it should be re-enabled.
  • It is not clear if the Install dependencies step is meant to replace the commented out Build WasmEdge with Release mode step or if it needs to be executed before it.

Overall, it seems that the main intention of this patch is to update the workflow by adding an Install dependencies step and an Install Rust-nightly step. Some clarification is needed regarding the commented out step and the ordering of the steps.

@apepkuss apepkuss requested a review from hydai November 15, 2023 11:01
@apepkuss
Copy link
Collaborator Author

@hydai Could you please help review this PR? Please ignore the failed rust-static-lib workflow. We'll fix it in another PR. Thanks a lot!

@apepkuss
Copy link
Collaborator Author

@hydai Thanks for the review!

@apepkuss apepkuss merged commit 06e7d2d into WasmEdge:main Nov 15, 2023
34 of 37 checks passed
@apepkuss apepkuss deleted the update-rustdoc branch November 16, 2023 14:39
apepkuss added a commit that referenced this pull request Nov 28, 2023
* [Refactor] Add #[form] for subtypes of WasmEdgeError.

Signed-off-by: csh <[email protected]>

* [Refactor] fix memory leak

Signed-off-by: csh <[email protected]>

* [Fix] Avoid the CPU 100% by tokio-wasi.

Signed-off-by: csh <[email protected]>

* [Fix] Fix blocking during tokio-wasi TCP connect.

Signed-off-by: csh <[email protected]>

* [Fix] async-wasi get envs & args

Signed-off-by: csh <[email protected]>

* [Perf] Optimize the wasi-ctx

Signed-off-by: csh <[email protected]>

* [Chore] Temporarily disable the serialize feature.

Signed-off-by: csh <[email protected]>

* [feat] virtual file system

Signed-off-by: csh <[email protected]>

* [Refactor] update the argument type of `run_func_with_timeout` and `run_func_async_with_timeout` (#76)

* [Feat] run func timeout use std::time::Duration

Signed-off-by: csh <[email protected]>

* [Doc]: update api docs

Signed-off-by: csh <[email protected]>

---------

Signed-off-by: csh <[email protected]>
(cherry picked from commit 07e84b7)
Signed-off-by: csh <[email protected]>

* [Refactor] delete useless code from validator

Signed-off-by: csh <[email protected]>

* [Refactor] To pass the clippy

Signed-off-by: csh <[email protected]>

* Disable timeout in musl libc (#71)

Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: csh <[email protected]>

* Disable timeout in musl libc

Signed-off-by: csh <[email protected]>

* [Doc] update doc

Signed-off-by: csh <[email protected]>

* [Fix] fix ffi::WasmEdge_String to String

Signed-off-by: csh <[email protected]>

* [Fix] Fix unit test

Signed-off-by: csh <[email protected]>

* [Fix] Modify the `WasiModule`.

Signed-off-by: csh <[email protected]>

* [refactor] Introduce `WasmEdge_FunctionInstanceGetData` to drop host data (#84)

* feat(rust-sys): drop host_data in `Function::drop`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update `drop` of `ImportModule` and `Function`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): update test code

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update `drop` of `ImportModule` and `Function`

Signed-off-by: Xin Liu <[email protected]>

* refactor(rust-sys): update `ImportModule::drop`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): remove debug code

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): remove debug code

Signed-off-by: Xin Liu <[email protected]>

* version(rust-sdk): bump to `0.13.0`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): supress clippy warning

Signed-off-by: Xin Liu <[email protected]>

* ci(ci-build): update rust version

Signed-off-by: Xin Liu <[email protected]>

* ci(standalone): update rust version

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update rustdoc

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): update rustdoc and `README`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update dependencies

Signed-off-by: Xin Liu <[email protected]>

* ci: update to `macos-13`

Signed-off-by: Xin Liu <[email protected]>

---------

Signed-off-by: Xin Liu <[email protected]>

* doc: update `CHANGELOG` (#85)

Signed-off-by: Xin Liu <[email protected]>

* [Chore] Update build script (#86)

* chore(rust-sys): update build script

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update build script

Signed-off-by: Xin Liu <[email protected]>

---------

Signed-off-by: Xin Liu <[email protected]>

* [Fix] Fix sys test

Signed-off-by: csh <[email protected]>

* Relax the version of `wat` dep  (#90)

* chore(rust-sdk): update `wat` dep

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): update test code

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update test code

Signed-off-by: Xin Liu <[email protected]>

* version(rust-sys): bump to `0.17.4`

Signed-off-by: Xin Liu <[email protected]>

* version(rust-sdk): bump to `0.13.1`

Signed-off-by: Xin Liu <[email protected]>

* doc: update `CHANGELOG`

Signed-off-by: Xin Liu <[email protected]>

---------

Signed-off-by: Xin Liu <[email protected]>

* [CI] skip test_vmbuilder on fedora

Signed-off-by: csh <[email protected]>

* Fix static build to link agains zstd (#91)

Signed-off-by: Jorge Prendes <[email protected]>

* Update doc for releasing `v0.13.2` (#93)

* ci: update workflows

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update versioning table

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): update versioning table

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): fix typo

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): update workflow

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update rustdoc

Signed-off-by: Xin Liu <[email protected]>

* version(rust-sdk): bump to `0.13.2`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): update versioning table

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): update workflow

Signed-off-by: Xin Liu <[email protected]>

---------

Signed-off-by: Xin Liu <[email protected]>

* [Version] Bump `wasmedge-sys` to `v0.17.5` (#94)

* version(rust-sys): bump to `0.17.5`

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update versioning table

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sys): update dependency

Signed-off-by: Xin Liu <[email protected]>

* chore(rust-sdk): update versioning table

Signed-off-by: Xin Liu <[email protected]>

---------

Signed-off-by: Xin Liu <[email protected]>

* doc: update `CHANGELOG` (#95)

Signed-off-by: Xin Liu <[email protected]>

* [CI] Update `rust-static-lib` workflow (#96)

* ci(rust-static-lib): update workflow

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): update workflow

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): update workflow

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): update workflow

Signed-off-by: Xin Liu <[email protected]>

* ci(rust-static-lib): remove `libboost-all-dev` dep

Signed-off-by: Xin Liu <[email protected]>

---------

Signed-off-by: Xin Liu <[email protected]>

* [Fix] Temporary FuncRef extraction implementation.

Signed-off-by: csh <[email protected]>

* [CI] fix test wat

Signed-off-by: csh <[email protected]>

---------

Signed-off-by: csh <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
Co-authored-by: Jorge Prendes <[email protected]>
Co-authored-by: Xin Liu <[email protected]>
@L-jasmine L-jasmine mentioned this pull request Dec 11, 2023
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