chore(actions) + refactor(lint): include linting in workflow

Also fixes various lint warnings/errors by selene.

.github/workflows/ci.yaml

@@ -1,43 +1,46 @@
 name: CI
 
 on:
-     push:
+  push:
-     pull_request:
+  pull_request:
-     workflow_dispatch:
+  workflow_dispatch:
 
 defaults:
-     run:
+  run:
-          shell: bash
+    shell: bash
 
 jobs:
-     fmt:
+  fmt:
-          runs-on: ubuntu-latest
+    runs-on: ubuntu-latest
-          steps:
+    steps:
-               - name: Checkout repository
+      - name: Checkout repository
-                 uses: actions/checkout@v4
+        uses: actions/checkout@v4
 
-               - name: Install tooling
+      - name: Install tooling
-                 uses: ok-nick/setup-aftman@v0.4.2
+        uses: ok-nick/setup-aftman@v0.4.2
-                 with:
+        with:
-                      cache: true
+          cache: true
 
-               - name: Check formatting
+      - name: Check formatting
-                 run: stylua -c -v .
+        run: stylua -c -v .
 
-     lint:
+  lint:
-          needs: ["fmt"]
+    needs: ["fmt"]
-          runs-on: ubuntu-latest
+    runs-on: ubuntu-latest
-          steps:
+    steps:
-               - name: Checkout repository
+      - name: Checkout repository
-                 uses: actions/checkout@v4
+        uses: actions/checkout@v4
 
-               - name: Install dependencies
+      - name: Install dependencies
-                 run: wally install
+        run: wally install
 
-               - name: Install tooling
+      - name: Install tooling
-                 uses: ok-nick/setup-aftman@v0.4.2
+        uses: ok-nick/setup-aftman@v0.4.2
-                 with:
+        with:
-                      cache: true
+          cache: true
 
-               - name: Analyze
+      - name: Analyze
-                 run: luau-lsp analyze --ignore="Packages/**" --settings=".vscode/settings.json" lib/ examples/ mod.luau
+        run: luau-lsp analyze --ignore="Packages/**" --settings=".vscode/settings.json" lib/ examples/ mod.luau
+
+      - name: lint
+        run: selene .

aftman.toml

@@ -4,3 +4,4 @@ stylua = "JohnnyMorganz/StyLua@0.20.0"
 luau-lsp = "JohnnyMorganz/luau-lsp@1.27.0"
 darklua = "seaofvoices/darklua@0.13.0"
 wally = "UpliftGames/wally@0.3.2"
+selene = "Kampfkarren/selene@0.26.1"

lib/future.luau

@@ -80,7 +80,7 @@ export type Future<T> = typeof(Future) & {
 	_status: Status,
 }
 
-local function _constructor<T>(fn: (Signal<()>, Signal<T, Status>) -> (), args: { any })
+local function _constructor<T>(fn: (Signal<()>, Signal<T, Status>) -> ())
 	return setmetatable(
 		{
 			_thread = coroutine.create(fn),
@@ -115,7 +115,7 @@ function Future.new<T>(fn: (...any) -> T, args: { any })
 
 		local ret = fn(table.unpack(args))
 		retEvt:Fire(ret, "ready")
-	end, args)
+	end)
 end
 
 --[=[
@@ -132,9 +132,9 @@ function Future.try<T>(fn: (...any) -> T, args: { any })
 		spawnEvt:Fire()
 
 		local ok, ret = pcall(fn, table.unpack(args))
-		local result: Result<T, string> = if ok then Ok(ret) else Err(ret)
+		local res: Result<T, string> = if ok then Ok(ret) else Err(ret)
-		retEvt:Fire(result, "ready")
+		retEvt:Fire(res, "ready")
-	end, args)
+	end)
 end
 
 --[=[

lib/result.luau

@@ -453,7 +453,8 @@ function Result.inspectErr<T, E>(self: Result<T, E>, op: (val: E) -> ()): Result
 end
 
 -- TODO: Iterator traits
-function Result.iter(): never
+-- selene: allow(unused_variable)
+function Result.iter<T, E>(self: Result<T, E>): never
 	return error("Unimplemented: `Result:iter()`")
 end
 
@@ -540,6 +541,7 @@ function Result.unwrap<T, E>(self: Result<T, E>): T | never
 end
 
 -- TODO: default values for types
+-- selene: allow(unused_variable)
 function Result.unwrapOrDefault<T, E>(self: Result<T, E>): never
 	return error("Unimplemented: `Result:unwrapOrDefault()`")
 end
@@ -596,9 +598,11 @@ function Result.unwrapErr<T, E>(self: Result<T, E>): E | never
 end
 
 -- TODO: How the fuck do I implement this?
+-- selene: allow(unused_variable)
 function Result.intoOk<T, E>(self: Result<T, E>): never
 	return error("Unimplemented: `Result:intoOk()`")
 end
+-- selene: allow(unused_variable)
 function Result.intoErr<T, E>(self: Result<T, E>): never
 	return error("Unimplemented: `Result:intoErr()`")
 end

selene.toml

@@ -0,0 +1,2 @@
+std = "luau"
+exclude = ["Packages/*"]