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

[CMake] Merge and move the simdjson fetching. #3426

Merged
merged 2 commits into from
May 27, 2024
Merged

Conversation

q82419
Copy link
Collaborator

@q82419 q82419 commented May 24, 2024

No description provided.

Copy link
Member

juntao commented May 24, 2024

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


@github-actions github-actions bot added c-Plugin An issue related to WasmEdge Plugin c-Test An issue/PR to enhance the test suite WASI-NN https://github.com/WebAssembly/wasi-nn c-CMake An issue related to CMake mechanism or options labels May 24, 2024
Copy link

codecov bot commented May 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.84%. Comparing base (25cec02) to head (baca7b1).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3426   +/-   ##
=======================================
  Coverage   79.84%   79.84%           
=======================================
  Files         253      253           
  Lines       34948    34948           
  Branches     6140     6141    +1     
=======================================
  Hits        27905    27905           
- Misses       5614     5623    +9     
+ Partials     1429     1420    -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@q82419 q82419 marked this pull request as ready for review May 24, 2024 10:58
@q82419 q82419 requested a review from ibmibmibm as a code owner May 24, 2024 10:58
@hydai
Copy link
Member

hydai commented May 24, 2024

@q82419 Windows failed.

Copy link
Member

@hydai hydai left a comment

Choose a reason for hiding this comment

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

Windows failed

@q82419 q82419 force-pushed the yiying/cmake_simdjson branch 2 times, most recently from e6b233e to a2e2aa4 Compare May 27, 2024 08:22
Signed-off-by: YiYing He <yiying@secondstate.io>
@hydai hydai merged commit fc2f3a7 into master May 27, 2024
87 of 88 checks passed
@hydai hydai deleted the yiying/cmake_simdjson branch May 27, 2024 10:23
dm4 pushed a commit that referenced this pull request Jul 2, 2024
Signed-off-by: YiYing He <yiying@secondstate.io>
dm4 pushed a commit that referenced this pull request Jul 2, 2024
Signed-off-by: YiYing He <yiying@secondstate.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-CMake An issue related to CMake mechanism or options c-Plugin An issue related to WasmEdge Plugin c-Test An issue/PR to enhance the test suite v-0.14.1 WASI-NN https://github.com/WebAssembly/wasi-nn
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants