BREAKING: Update to the latest ZLS master

Rule runners now use `zlinter.rules.RunError` for extendability and coverage of IO related errors

Author: KurtWagner

build.zig.zon

@@ -7,8 +7,8 @@
         .zls = .{
             // Update with `zig fetch --save git+https://github.com/zigtools/zls#master`
             // IF changed THEN update .github/coverage.yml
-            .url = "git+https://github.com/zigtools/zls?ref=master#d03b99209afcae2169912986b2f4c677fca279c9",
-            .hash = "zls-0.16.0-dev-rmm5fu5GJQBnXt4idjOYx7p4104rd3Ji-daO8a-7Gx6e",
+            .url = "git+https://github.com/zigtools/zls?ref=master#be4ddd69067e1fd62644d9a999b17adffd8766c2",
+            .hash = "zls-0.16.0-dev-rmm5flNfJQA26cb4HX9zW4379iE2lPGPDZD0qdh_VXMA",
         },
     },
     .paths = .{

integration_tests/src/no_cats.zig

@@ -20,7 +20,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
     if (config.severity == .off) return null;
 

src/lib/rules.zig

@@ -7,7 +7,7 @@ pub const LintRule = struct {
         doc: *const session.LintDocument,
         gpa: std.mem.Allocator,
         options: RunOptions,
-    ) error{OutOfMemory}!?results.LintResult,
+    ) RunError!?results.LintResult,
 };
 
 pub const RunOptions = struct {
@@ -19,6 +19,8 @@ pub const RunOptions = struct {
     }
 };
 
+pub const RunError = std.mem.Allocator.Error || std.Io.Cancelable;
+
 /// Rules the modify the execution of rules.
 pub const RuleOptions = struct {}; // zlinter-disable-current-line
 

src/lib/session.zig

@@ -256,7 +256,7 @@ pub const LintContext = struct {
             buffer[0..size],
         );
 
-        const handle = self.document_store.getOrLoadHandle(uri) orelse return error.HandleError;
+        const handle = (try self.document_store.getOrLoadHandle(uri)) orelse return error.HandleError;
 
         var src_comments = try comments.allocParse(handle.tree.source, gpa);
         errdefer src_comments.deinit(gpa);

src/rules/declaration_naming.zig

@@ -94,7 +94,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
 
     var lint_problems = std.ArrayList(zlinter.results.LintProblem).empty;

src/rules/field_naming.zig

@@ -145,7 +145,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
 
     var lint_problems: std.ArrayList(zlinter.results.LintProblem) = .empty;

src/rules/field_ordering.zig

@@ -52,7 +52,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
 
     var lint_problems = std.ArrayList(zlinter.results.LintProblem).empty;

src/rules/file_naming.zig

@@ -33,7 +33,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
 
     const message, const severity = msg: {

src/rules/function_naming.zig

@@ -69,7 +69,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
 
     var lint_problems = std.ArrayList(zlinter.results.LintProblem).empty;

src/rules/import_ordering.zig

@@ -58,7 +58,7 @@ fn run(
     doc: *const zlinter.session.LintDocument,
     gpa: std.mem.Allocator,
     options: zlinter.rules.RunOptions,
-) error{OutOfMemory}!?zlinter.results.LintResult {
+) zlinter.rules.RunError!?zlinter.results.LintResult {
     const config = options.getConfig(Config);
     if (config.severity == .off) return null;