Skip to content

Conversation

@CjiW
Copy link
Contributor

@CjiW CjiW commented Dec 21, 2022

token 分为 wordsymbol
修复:

  • 捕获错误
  • 错误容忍

@CjiW CjiW requested a review from Chronostasys December 21, 2022 13:46
@CjiW CjiW linked an issue Dec 21, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Dec 21, 2022

Codecov Report

Merging #155 (dc10277) into master (ac76010) will decrease coverage by 0.81%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
- Coverage   88.63%   87.82%   -0.82%     
==========================================
  Files          64       65       +1     
  Lines       10930    11035     +105     
==========================================
+ Hits         9688     9691       +3     
- Misses       1242     1344     +102     
Impacted Files Coverage Δ
src/nomparser/array.rs 81.25% <100.00%> (+4.32%) ⬆️
src/nomparser/comment.rs 100.00% <100.00%> (ø)
src/nomparser/constval.rs 100.00% <100.00%> (ø)
src/nomparser/control.rs 97.84% <100.00%> (+0.18%) ⬆️
src/nomparser/expression.rs 99.51% <100.00%> (+<0.01%) ⬆️
src/nomparser/function.rs 98.96% <100.00%> (+0.01%) ⬆️
src/nomparser/helper.rs 85.89% <100.00%> (+2.82%) ⬆️
src/nomparser/identifier.rs 96.73% <100.00%> (+0.10%) ⬆️
src/nomparser/implement.rs 98.57% <100.00%> (+0.18%) ⬆️
src/nomparser/pkg.rs 100.00% <100.00%> (ø)
... and 13 more

@CjiW CjiW requested a review from RINNE-TAN December 21, 2022 13:56
Copy link
Member

@Chronostasys Chronostasys left a comment

Choose a reason for hiding this comment

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

LGTM!

@CjiW CjiW merged commit 86941f1 into master Dec 27, 2022
@Chronostasys Chronostasys deleted the fix/space branch December 27, 2022 10:54
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