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

Fix for clippy 1.73 #995

Merged
merged 1 commit into from
Oct 5, 2023
Merged

Fix for clippy 1.73 #995

merged 1 commit into from
Oct 5, 2023

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Oct 5, 2023

https://github.com/rust-lang/rust/blob/master/RELEASES.md was was released today and clippy learned some new tricks

@coveralls
Copy link

Pull Request Test Coverage Report for Build 6423547967

  • 5 of 8 (62.5%) changed or added relevant lines in 2 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.03%) to 87.269%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/parser/mod.rs 4 7 57.14%
Files with Coverage Reduction New Missed Lines %
src/parser/mod.rs 1 82.69%
src/lib.rs 2 12.28%
Totals Coverage Status
Change from base Build 6383573174: 0.03%
Covered Lines: 16650
Relevant Lines: 19079

💛 - Coveralls

@alamb
Copy link
Contributor Author

alamb commented Oct 5, 2023

Merging this in to get clean CI runs

@alamb alamb merged commit 02f3d78 into apache:main Oct 5, 2023
10 checks passed
@alamb alamb deleted the alamb/fix_clippy2 branch October 5, 2023 19:25
MazterQyou pushed a commit to cube-js/sqlparser-rs that referenced this pull request Dec 19, 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.

2 participants