Skip to content

Commit 12242f4

Browse files
Copilotl0lawrence
andcommitted
Add --isolate flag to all azpysdk calls in skill files
Co-authored-by: l0lawrence <100643745+l0lawrence@users.noreply.github.com>
1 parent 80ad9ce commit 12242f4

3 files changed

Lines changed: 22 additions & 22 deletions

File tree

.github/skills/fix-black/SKILL.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ Intelligently fixes black formatting issues by:
2323

2424
**Command for entire package:**
2525
```powershell
26-
python -m azpysdk black --pkg-path <package-path>
26+
python -m azpysdk --isolate black --pkg-path <package-path>
2727
```
2828

2929
**Command for specific file/module:**
3030
```powershell
31-
python -m azpysdk black --pkg-path <package-path> -- path/to/file.py
31+
python -m azpysdk --isolate black --pkg-path <package-path> -- path/to/file.py
3232
```
3333

3434
## Reference Documentation
@@ -92,13 +92,13 @@ Black is an auto-formatter — it rewrites files to comply with the style. Simpl
9292
**Format entire package:**
9393
```powershell
9494
# Run black on the entire package (within activated venv)
95-
python -m azpysdk black --pkg-path <package-path>
95+
python -m azpysdk --isolate black --pkg-path <package-path>
9696
```
9797

9898
**Format specific files (if issue mentions specific files):**
9999
```powershell
100100
# Run black on specific files (within activated venv)
101-
python -m azpysdk black --pkg-path <package-path> -- path/to/specific_file.py
101+
python -m azpysdk --isolate black --pkg-path <package-path> -- path/to/specific_file.py
102102
```
103103

104104
### Step 4: Review Changes
@@ -119,7 +119,7 @@ Re-run black in check mode to confirm all files are now properly formatted:
119119

120120
```powershell
121121
# Verify formatting (within activated venv)
122-
python -m azpysdk black --pkg-path <package-path>
122+
python -m azpysdk --isolate black --pkg-path <package-path>
123123
```
124124

125125
If black reports no files were changed, all formatting issues are resolved.
@@ -194,7 +194,7 @@ The Azure SDK for Python uses a repo-wide black configuration located at `eng/bl
194194
- Line length: 120 characters
195195
- Target Python versions: 3.8+
196196

197-
The `python -m azpysdk black` command automatically uses this configuration, so no manual configuration is needed.
197+
The `python -m azpysdk --isolate black` command automatically uses this configuration, so no manual configuration is needed.
198198

199199
## Common Black Formatting Changes
200200

@@ -225,14 +225,14 @@ pip install -r dev_requirements.txt
225225
pip install -e .
226226
227227
# 2. Run black to auto-format the package
228-
python -m azpysdk black --pkg-path sdk/storage/azure-storage-blob
228+
python -m azpysdk --isolate black --pkg-path sdk/storage/azure-storage-blob
229229
230230
# 3. Review what changed
231231
git diff --name-only
232232
git diff
233233
234234
# 4. Verify no remaining issues
235-
python -m azpysdk black --pkg-path sdk/storage/azure-storage-blob
235+
python -m azpysdk --isolate black --pkg-path sdk/storage/azure-storage-blob
236236
237237
# 5. Create PR referencing the issue
238238
$branchName = "style/azure-storage-blob-black-99999"

.github/skills/fix-mypy/SKILL.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ Intelligently fixes mypy issues by:
2626

2727
**Command for entire package:**
2828
```powershell
29-
python -m azpysdk mypy --pkg-path <package-path>
29+
python -m azpysdk --isolate mypy --pkg-path <package-path>
3030
```
3131

3232
**Command for specific file/module:**
3333
```powershell
34-
python -m azpysdk mypy --pkg-path <package-path> -- path/to/file.py
34+
python -m azpysdk --isolate mypy --pkg-path <package-path> -- path/to/file.py
3535
```
3636

3737
## Reference Documentation
@@ -98,13 +98,13 @@ Based on the GitHub issue details, determine which files to check:
9898
**Option A - Issue specifies files:**
9999
```powershell
100100
# Run mypy on specific files mentioned in the issue (within activated venv)
101-
python -m azpysdk mypy --pkg-path <package-path> -- path/to/specific_file.py
101+
python -m azpysdk --isolate mypy --pkg-path <package-path> -- path/to/specific_file.py
102102
```
103103

104104
**Option B - Issue mentions module/directory:**
105105
```powershell
106106
# Run mypy on entire package (within activated venv)
107-
python -m azpysdk mypy --pkg-path <package-path> -- azure/specific/module/
107+
python -m azpysdk --isolate mypy --pkg-path <package-path> -- azure/specific/module/
108108
```
109109

110110
**Option C - Check modified files (if no specific target):**
@@ -119,7 +119,7 @@ git diff --cached --name-only | Select-String "<package-path>"
119119

120120
```powershell
121121
# Run mypy targeting the specific area from the issue (within activated venv)
122-
python -m azpysdk mypy --pkg-path <package-path> -- <target-from-issue>
122+
python -m azpysdk --isolate mypy --pkg-path <package-path> -- <target-from-issue>
123123
```
124124

125125
### Step 5: Analyze Type Errors
@@ -342,7 +342,7 @@ pip install -e .
342342
$targetFile = "azure/storage/blob/_blob_client.py"
343343
344344
# 3. Run mypy on specific file
345-
python -m azpysdk mypy --pkg-path sdk/storage/azure-storage-blob -- $targetFile
345+
python -m azpysdk --isolate mypy --pkg-path sdk/storage/azure-storage-blob -- $targetFile
346346
347347
# 4. Analyze output and identify fixable issues
348348
# Cross-reference with GitHub issue #67890
@@ -353,7 +353,7 @@ grep -r "from typing import" azure/storage/blob/ | findstr "_blob_client"
353353
# 6. Apply fixes to identified files
354354
355355
# 7. Re-run mypy to verify
356-
python -m azpysdk mypy --pkg-path sdk/storage/azure-storage-blob -- $targetFile
356+
python -m azpysdk --isolate mypy --pkg-path sdk/storage/azure-storage-blob -- $targetFile
357357
358358
# 8. Report results
359359

.github/skills/fix-pylint/SKILL.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ Intelligently fixes pylint issues by:
2626

2727
**Command for entire package:**
2828
```powershell
29-
python -m azpysdk pylint --pkg-path <package-path>
29+
python -m azpysdk --isolate pylint --pkg-path <package-path>
3030
```
3131

3232
**Command for specific file/module:**
3333
```powershell
34-
python -m azpysdk pylint --pkg-path <package-path> -- path/to/file.py
34+
python -m azpysdk --isolate pylint --pkg-path <package-path> -- path/to/file.py
3535
```
3636

3737
## Reference Documentation
@@ -96,13 +96,13 @@ Based on the GitHub issue details, determine which files to check:
9696
**Option A - Issue specifies files:**
9797
```powershell
9898
# Run pylint on specific files mentioned in the issue (within activated venv)
99-
python -m azpysdk pylint --pkg-path <package-path> -- path/to/specific_file.py
99+
python -m azpysdk --isolate pylint --pkg-path <package-path> -- path/to/specific_file.py
100100
```
101101

102102
**Option B - Issue mentions module/directory:**
103103
```powershell
104104
# Run pylint on entire package (within activated venv)
105-
python -m azpysdk pylint --pkg-path <package-path> -- azure/specific/module/
105+
python -m azpysdk --isolate pylint --pkg-path <package-path> -- azure/specific/module/
106106
```
107107

108108
**Option C - Check modified files (if no specific target):**
@@ -117,7 +117,7 @@ git diff --cached --name-only | Select-String "<package-path>"
117117

118118
```powershell
119119
# Run pylint targeting the specific area from the issue (within activated venv)
120-
python -m azpysdk pylint --pkg-path <package-path> -- <target-from-issue>
120+
python -m azpysdk --isolate pylint --pkg-path <package-path> -- <target-from-issue>
121121
```
122122

123123
### Step 5: Analyze Warnings
@@ -291,7 +291,7 @@ pip install -e .
291291
$targetFile = "azure/storage/blob/_blob_client.py"
292292
293293
# 3. Run pylint on specific file
294-
python -m azpysdk pylint --pkg-path sdk/storage/azure-storage-blob -- $targetFile
294+
python -m azpysdk --isolate pylint --pkg-path sdk/storage/azure-storage-blob -- $targetFile
295295
296296
# 4. Analyze output and identify fixable issues
297297
# Cross-reference with GitHub issue #12345
@@ -302,7 +302,7 @@ grep -r "similar_pattern" azure/storage/blob/
302302
# 6. Apply fixes to identified files
303303
304304
# 7. Re-run pylint to verify
305-
python -m azpysdk pylint --pkg-path sdk/storage/azure-storage-blob -- $targetFile
305+
python -m azpysdk --isolate pylint --pkg-path sdk/storage/azure-storage-blob -- $targetFile
306306
307307
# 8. Report results
308308

0 commit comments

Comments
 (0)