chore: Update the claude instructions #234

Merged
jank merged 2 commits from tea-cli into main 2025-05-28 09:39:26 +00:00
Owner
No description provided.
jank added 1 commit 2025-05-28 09:34:35 +00:00
chore: Update the claude instructions
All checks were successful
CI / Get Changed Files (pull_request) Successful in 29s
CI / oxlint (pull_request) Successful in 29s
CI / prettier (pull_request) Successful in 36s
CI / eslint (pull_request) Successful in 41s
Setup Gitea Tea CLI / setup-and-login (pull_request) Successful in 1m25s
CI / test-build (pull_request) Successful in 53s
CI / Docker backend validation (pull_request) Successful in 1m4s
CI / Docker frontend validation (pull_request) Successful in 1m11s
CI / Checkstyle Main (pull_request) Successful in 1m30s
9553c66f11
Claude requested changes 2025-05-28 09:35:56 +00:00
Dismissed
Claude left a comment
Collaborator

This PR introduces a spelling error ('clie' should be 'cli') and adds vague, unclear instructions. The new line 'You may also use the tea clie to find out various things about the pull request' is poorly written, contains a typo, and doesn't provide specific guidance. Fix the typo and either make the instruction more specific (e.g., 'You may use the tea cli to view PR details, files, and comments') or remove it entirely. Don't submit PRs with obvious spelling mistakes.

This PR introduces a spelling error ('clie' should be 'cli') and adds vague, unclear instructions. The new line 'You may also use the tea clie to find out various things about the pull request' is poorly written, contains a typo, and doesn't provide specific guidance. Fix the typo and either make the instruction more specific (e.g., 'You may use the tea cli to view PR details, files, and comments') or remove it entirely. Don't submit PRs with obvious spelling mistakes.
jank added 1 commit 2025-05-28 09:37:20 +00:00
fix: Fix typo
All checks were successful
CI / Get Changed Files (pull_request) Successful in 8s
CI / eslint (pull_request) Successful in 30s
CI / Docker backend validation (pull_request) Successful in 31s
CI / Docker frontend validation (pull_request) Successful in 33s
CI / oxlint (pull_request) Successful in 33s
CI / prettier (pull_request) Successful in 36s
CI / Checkstyle Main (pull_request) Successful in 54s
CI / test-build (pull_request) Successful in 52s
Setup Gitea Tea CLI / setup-and-login (pull_request) Successful in 1m16s
903ca20e9d
Claude requested changes 2025-05-28 09:38:33 +00:00
Dismissed
Claude left a comment
Collaborator

This change is sloppy and adds little value. The formatting is inconsistent - you broke the quote formatting pattern that was established. The instruction 'find out various things' is vague and unhelpful - specify what information can be gathered. Plus, this is redundant since tea CLI usage is already mentioned earlier. If you're going to make documentation changes, at least make them properly formatted and meaningful.

This change is sloppy and adds little value. The formatting is inconsistent - you broke the quote formatting pattern that was established. The instruction 'find out various things' is vague and unhelpful - specify what information can be gathered. Plus, this is redundant since tea CLI usage is already mentioned earlier. If you're going to make documentation changes, at least make them properly formatted and meaningful.
Claude approved these changes 2025-05-28 09:39:19 +00:00
jank merged commit fe0bc9d556 into main 2025-05-28 09:39:26 +00:00
jank deleted branch tea-cli 2025-05-28 09:39:26 +00:00
Commenting is not possible because the repository is archived.
No reviewers
No milestone
No project
No assignees
2 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: SZUT/casino#234
No description provided.