readied

Author: juleswritescode

crates/pgls_completions/src/relevance/scoring.rs

@@ -87,7 +87,8 @@ impl CompletionScore<'_> {
                 WrappingClause::Delete => 10,
                 WrappingClause::From => 5,
                 WrappingClause::Join { on_node }
-                    if on_node.is_none_or(|on| ctx.node_under_cursor.end_byte() < on.start_byte()) =>
+                    if on_node
+                        .is_none_or(|on| ctx.node_under_cursor.end_byte() < on.start_byte()) =>
                 {
                     5
                 }
@@ -106,7 +107,8 @@ impl CompletionScore<'_> {
                 WrappingClause::Where => 10,
                 WrappingClause::CheckOrUsingClause => 0,
                 WrappingClause::Join { on_node }
-                    if on_node.is_some_and(|on| ctx.node_under_cursor.start_byte() > on.end_byte()) =>
+                    if on_node
+                        .is_some_and(|on| ctx.node_under_cursor.start_byte() > on.end_byte()) =>
                 {
                     // Users will probably join on primary keys
                     if col.is_primary_key { 20 } else { 10 }

crates/pgls_completions/src/test_helper.rs

@@ -199,7 +199,7 @@ pub(crate) async fn assert_no_complete_results(query: &str, setup: Option<&str>,
     let params = get_test_params(&tree, &cache, query.into());
     let items = complete(params);
 
-    println!("{:#?}", items);
+    println!("{items:#?}");
 
     assert_eq!(items.len(), 0)
 }

crates/pgls_treesitter/src/context/ancestors.rs

@@ -3,7 +3,7 @@ pub struct Scope {
     pub ancestors: AncestorTracker,
 }
 
-static SCOPE_BOUNDARIES: &[&'static str] = &["statement", "ERROR", "program"];
+static SCOPE_BOUNDARIES: &[&str] = &["statement", "ERROR", "program"];
 
 #[derive(Debug)]
 pub struct ScopeTracker {
@@ -22,7 +22,7 @@ impl ScopeTracker {
 
         self.scopes
             .last_mut()
-            .expect(format!("Unhandled node kind: {}", node.kind()).as_str())
+            .unwrap_or_else(|| panic!("Unhandled node kind: {}", node.kind()))
             .ancestors
             .register(node, position);
     }
@@ -86,7 +86,7 @@ impl AncestorTracker {
     }
 
     pub fn history_ends_with(&self, matchers: &[&'static str]) -> bool {
-        assert!(matchers.len() > 0);
+        assert!(!matchers.is_empty());
 
         let mut tracking_idx = matchers.len() - 1;