Skip to content

fix(docs): correct skip:test:long_running label name in CLAUDE.md [PYSDK-89] #577

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
helmut-hoffer-von-ankershoffen merged 1 commit into from
Apr 24, 2026
Merged

fix(docs): correct skip:test:long_running label name in CLAUDE.md [PYSDK-89] #577

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ If you write code yourself, it is a strict requirement to validate your work on

If you you are creating a pull request yourself:
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line 33 has a duplicated word: "If you you are creating a pull request yourself" → "If you are creating a pull request yourself".

Suggested change
If you you are creating a pull request yourself:
If you are creating a pull request yourself:

Copilot uses AI. Check for mistakes.

* Add a label skip:test_long_running, to skip running long running tests. This is important because some tests in this repository are marked as long_running and can take a significant amount of time to complete. By adding this label, you help ensure that the CI pipeline runs efficiently and avoids unnecessary delays.
* Add a label skip:test:long_running, to skip running long running tests. This is important because some tests in this repository are marked as long_running and can take a significant amount of time to complete. By adding this label, you help ensure that the CI pipeline runs efficiently and avoids unnecessary delays.
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The label name on this bullet is plain text with punctuation attached ("skip:test:long_running,") which makes it easy to copy/paste incorrectly and is inconsistent with later usage (e.g., skip:test:long_running). Consider formatting the label as inline code and placing the comma outside the code span.

Suggested change
* Add a label skip:test:long_running, to skip running long running tests. This is important because some tests in this repository are marked as long_running and can take a significant amount of time to complete. By adding this label, you help ensure that the CI pipeline runs efficiently and avoids unnecessary delays.
* Add a label `skip:test:long_running`, to skip running long running tests. This is important because some tests in this repository are marked as long_running and can take a significant amount of time to complete. By adding this label, you help ensure that the CI pipeline runs efficiently and avoids unnecessary delays.

Copilot uses AI. Check for mistakes.

## Module Documentation Index

Expand Down Expand Up @@ -1163,7 +1163,7 @@ git push origin feat/my-feature
gh pr create --title "feat: add operation caching" --body "Description..."

# IMPORTANT: Add label to skip long-running tests
gh pr edit --add-label "skip:test_long_running"
gh pr edit --add-label "skip:test:long_running"
```

**PR triggers:**
Expand Down Expand Up @@ -1367,7 +1367,7 @@ git commit -m "docs: update README [skip ci]"
git commit -m "skip:ci: work in progress"

# Add PR label to skip long-running tests
gh pr edit --add-label "skip:test_long_running"
gh pr edit --add-label "skip:test:long_running"
```

### IDE Setup Recommendations
Expand Down
Loading