Skip to content

Commit 598c170

Browse files
committed
chore: fix linter warning
Signed-off-by: Christian Stewart <christian@aperture.us>
1 parent 0505c1c commit 598c170

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

AGENTS.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ Once the issue is fixed and the compliance test passes you may delete WIP.md wit
101101

102102
When working with golangci-lint:
103103

104-
1. **Running the Linter**: Use `golangci-lint run ./...` to check for linter errors
104+
1. **Running the Linter**: Use `yarn lint` to run the linter, `yarn lint:go` for go and `yarn lint:js` for js
105105
2. **Fixing Errors**: Address linter errors in the affected code files
106106
3. **Iterating**: Repeat the linting process until no errors remain
107107
4. **Ignoring Warnings**: You can ignore linter errors with inline comments when the warning is unnecessarily strict:
@@ -110,6 +110,8 @@ When working with golangci-lint:
110110
```
111111
This is appropriate for cases like deferring Close without checking the error return value, which can often be safely ignored.
112112

113+
Make sure that `yarn test` and `yarn lint` both pass before suggesting a task is complete.
114+
113115
## Specialized Workflows
114116

115117
### Squash Commits

gs/reflect/type.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import {
55
TypeKind,
66
isStructTypeInfo,
77
isInterfaceTypeInfo,
8-
type MethodSignature,
98
} from '../builtin/type.js'
109

1110
// rtype is the common implementation of most values

0 commit comments

Comments
 (0)