summaryrefslogtreecommitdiffhomepage
path: root/.github/workflows
diff options
context:
space:
mode:
authorDidier Durand <[email protected]>2025-12-27 07:46:06 +0100
committerGitHub <[email protected]>2025-12-27 00:46:06 -0600
commite6b9988fa4913d25fe2f6bf5b0f3b7218f8c8e7e (patch)
tree0aacf131938c4d4fdf2dd604aea144b34ba9b411 /.github/workflows
parent3c02d5d338698d382b02fbbf7eaa07a9b0f1ebc6 (diff)
downloadopencode-e6b9988fa4913d25fe2f6bf5b0f3b7218f8c8e7e.tar.gz
opencode-e6b9988fa4913d25fe2f6bf5b0f3b7218f8c8e7e.zip
doc: fix typos in various files (#6238)
Diffstat (limited to '.github/workflows')
-rw-r--r--.github/workflows/review.yml2
1 files changed, 1 insertions, 1 deletions
diff --git a/.github/workflows/review.yml b/.github/workflows/review.yml
index c0e3a5deb..44bfeb336 100644
--- a/.github/workflows/review.yml
+++ b/.github/workflows/review.yml
@@ -64,7 +64,7 @@ jobs:
Please check all the code changes in this pull request against the style guide, also look for any bugs if they exist. Diffs are important but make sure you read the entire file to get proper context. Make it clear the suggestions are merely suggestions and the human can decide what to do
When critiquing code against the style guide, be sure that the code is ACTUALLY in violation, don't complain about else statements if they already use early returns there. You may complain about excessive nesting though, regardless of else statement usage.
- When critiquing code style don't be a zealot, we don't like "let" statements but sometimes they are the simpliest option, if someone does a bunch of nesting with let, they should consider using iife (see packages/opencode/src/util.iife.ts)
+ When critiquing code style don't be a zealot, we don't like "let" statements but sometimes they are the simplest option, if someone does a bunch of nesting with let, they should consider using iife (see packages/opencode/src/util.iife.ts)
Use the gh cli to create comments on the files for the violations. Try to leave the comment on the exact line number. If you have a suggested fix include it in a suggestion code block.
If you are writing suggested fixes, BE SURE THAT the change you are recommending is actually valid typescript, often I have seen missing closing "}" or other syntax errors.