Skip to content

Conversation

@YinY1
Copy link
Contributor

@YinY1 YinY1 commented Jun 14, 2025

No description provided.

@YinY1
Copy link
Contributor Author

YinY1 commented Jun 15, 2025

@wangl-cc 未通过的ci出现在maa-cli,与maa-sys无关。似乎是rustc或clippy更新引起更强的检查导致的,是否需要一并修改

@wangl-cc
Copy link
Member

@wangl-cc 未通过的ci出现在maa-cli,与maa-sys无关。似乎是rustc或clippy更新引起更强的检查导致的,是否需要一并修改

应该是新版的 cllipy 新加的lint,我晚上改一下,然后你 rebase一下应该就没问题了。不过我暂时没有windows设备,所以没办法验证,你如果本地测试没有问题,之后就可以合并了。

@codecov
Copy link

codecov bot commented Jun 15, 2025

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 61.55%. Comparing base (df612be) to head (1db57de).

Files with missing lines Patch % Lines
crates/maa-sys/src/lib.rs 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #404      +/-   ##
==========================================
- Coverage   61.64%   61.55%   -0.10%     
==========================================
  Files          42       42              
  Lines        5009     5018       +9     
  Branches     5009     5018       +9     
==========================================
+ Hits         3088     3089       +1     
- Misses       1663     1672       +9     
+ Partials      258      257       -1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wangl-cc
Copy link
Member

感谢贡献!

@wangl-cc wangl-cc merged commit 6cc716a into MaaAssistantArknights:main Jun 15, 2025
17 of 19 checks passed
@YinY1 YinY1 deleted the feat-ex-bindings branch June 15, 2025 15:02
@YinY1
Copy link
Contributor Author

YinY1 commented Jun 15, 2025

#398 抱歉忘记关联issue了,应该可以关闭这个issue

@wangl-cc
Copy link
Member

这个 PR 只在 maa-sys 部分增加了 extra,但是 maa-cli 里面还没有添加,用户目前还不能使用这个功能。所以那个 issue 可以保持打开,等到 maa-cli 也实现了之后再关掉。

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