Skip to content

Commit a2da1de

Browse files
committed
Address review feedback on python-environment skill
Shorten the frontmatter description, restore hard "never activate" rule to match AGENTS.md, and remove the conda/mamba env-store note. Made-with: Cursor
1 parent 3ec83a7 commit a2da1de

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

  • .github/skills/python-environment

.github/skills/python-environment/SKILL.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: python-environment
3-
description: Detect and configure a conda-compatible tool, reuse an existing CausalPy environment when possible, create it only when needed, and run commands inside it. Use before tasks that need the project environment, such as importing project code, running tests, building docs, or invoking repo tooling.
3+
description: Detect, configure, and use a conda-compatible tool. Use before tasks that need the project environment, such as importing project code, running tests, building docs, or invoking repo tooling.
44
---
55

66
# Python Environment
@@ -62,7 +62,7 @@ $CONDA_EXE run -n CausalPy make setup
6262

6363
## Run commands
6464

65-
Prefer `run -n` instead of `activate`:
65+
Never use `$CONDA_EXE activate`, instead use `$CONDA_EXE run -n CausalPy <command>`.
6666

6767
```bash
6868
$CONDA_EXE run -n CausalPy <command>
@@ -89,8 +89,6 @@ $CONDA_EXE run -p "/full/path/to/CausalPy" <command>
8989

9090
Keep using `run -p` with that full prefix for the rest of the session.
9191

92-
Note: a shell `conda` function may still resolve through the same libmamba installation as `mamba`, so switching from `mamba` to `conda` does not necessarily change which env store is searched.
93-
9492
### Git worktrees and remote machines
9593

9694
Git worktrees do not require a fresh env per agent session. Prefer reusing an existing env to save time. The main caveat is that this repo uses editable installs, so one shared env can point at whichever checkout most recently ran `make setup`.

0 commit comments

Comments
 (0)