Merge pull request #43 from yubing744/feature-ignore-script-warn

feat: test skip public(script) warning

language/move-analyzer/editors/code/tests/lsp-demo/Move.toml

@@ -2,8 +2,8 @@
 name = "Symbols"
 version = "0.0.1"
 
-[dependencies]
-MoveStdlib = { local = "../../../../../../move-stdlib/", addr_subst = { "std" = "0x1" } }
-
 [addresses]
 Symbols = "0xCAFE"
+
+[dependencies]
+StarcoinFramework = {git = "https://github.com/starcoinorg/starcoin-framework.git", rev="cf1deda180af40a8b3e26c0c7b548c4c290cd7e7"}

language/move-analyzer/src/symbols.rs

@@ -690,7 +690,22 @@ impl Symbolicator {
             // start with empty diagnostics for all files and replace them with actual diagnostics
             // only for files that have failures/warnings so that diagnostics for all other files
             // (that no longer have failures/warnings) are reset
-            ide_diagnostics.extend(lsp_diagnostics);
+            for (key, value) in lsp_diagnostics {
+                let mut diags:Vec<Diagnostic> = vec![];
+
+                for diag in value {
+                    if diag.message.contains("'public(script)' is deprecated") { // skip public(script) warning for starcoin chain
+                        continue;
+                    }
+
+                    diags.push(diag);
+                }
+
+                if diags.len() > 0 {
+                    ide_diagnostics.insert(key, diags);
+                }
+            }
+
             if failure {
                 // just return diagnostics as we don't have typed AST that we can use to compute
                 // symbolication information