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.

Sign up for GitHub

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

Jump to bottom

Stricter lints in Language Server #691

Merged
merged 23 commits into from
Aug 29, 2023
Merged

Stricter lints in Language Server #691

merged 23 commits into from
Aug 29, 2023

Conversation

JonasWanke
Copy link
Member

Checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

@JonasWanke
Fix clippy::must_use_candidate
3652cdc
@JonasWanke
Fix clippy::semicolon_if_nothing_returned
90e5cc1
@JonasWanke
Fix clippy::implicit_clone
c5984ab
@JonasWanke
Fix clippy::redundant_closure_for_method_calls
4ae493c
@JonasWanke
Fix clippy::unused_async
bbff41c
@JonasWanke
Fix clippy::unnecessary_wraps
c478460
@JonasWanke
Fix clippy::needless_pass_by_value
85cf6dc
@JonasWanke
Fix clippy::trivially_copy_pass_by_ref
9c8f9f0
@JonasWanke
Fix clippy::default_trait_access
9b5b66c
@JonasWanke
Fix clippy::map_unwrap_or
f5a10fd
@JonasWanke
Fix clippy::flat_map_option
8e9fac3
@JonasWanke
Fix clippy::manual_assert
60c1dee
@JonasWanke
Fix clippy::items_after_statements
1936fe7
@JonasWanke
Fix clippy::explicit_iter_loop
ea8058c
@JonasWanke
Fix clippy::cast_possible_truncation
7d1ceb8
@JonasWanke
Fix candy::case_sensitive_file_extension_comparisons
019f5f3
@JonasWanke
Fix clippy::doc_markdown
40884db
@JonasWanke
Fix clippy::redundant_closure_for_method_calls
c575a77
@JonasWanke
Fix clippy::unused_self
a7ec3a6
@JonasWanke
Fix clippy::use_self
fb13d5b
@JonasWanke
Fix clippy::option_if_let_else
cf991fc
@JonasWanke
Fix clippy::missing_const_for_fn
f9676d7
@JonasWanke
Enable clippy::nursery, ::pedantic
04462fb
@JonasWanke JonasWanke added the T: Refactor Type: Refactoring label Aug 29, 2023
@jwbot jwbot enabled auto-merge August 29, 2023 17:22
@jwbot jwbot added P: Compiler: Language Server Package: The Candy Language Server P: Compiler: Frontend Package: The compiler frontend labels Aug 29, 2023
@JonasWanke JonasWanke mentioned this pull request Aug 29, 2023
Open
@jwbot jwbot merged commit 623ce63 into main Aug 29, 2023
11 checks passed
@jwbot jwbot deleted the stricter-lints-in-language-server branch August 29, 2023 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarcelGarus MarcelGarus MarcelGarus approved these changes

Assignees

@JonasWanke JonasWanke

Labels
P: Compiler: Frontend Package: The compiler frontend P: Compiler: Language Server Package: The Candy Language Server T: Refactor Type: Refactoring
Projects
🍭 Candy
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JonasWanke @MarcelGarus @jwbot