Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ## Environment Setup | ||
|
|
||
| The documentation toolchain is defined in `devtools/conda-envs/docs_env.yaml`. Create (or update) the environment before running any docs-specific tasks: | ||
|
|
||
| ```bash | ||
| conda env create -f devtools/conda-envs/docs_env.yaml # first-time setup | ||
| conda env update -f devtools/conda-envs/docs_env.yaml # subsequent syncs | ||
| conda activate pyunitwizard-docs | ||
| ``` |
There was a problem hiding this comment.
Fix broken conda environment commands
The environment setup snippet uses conda env create -f devtools/conda-envs/docs_env.yaml and conda env update -f …, but the YAML file does not define an environment name. Running those commands raises CondaValueError: Key 'name' missing unless the user supplies a name or the helper scripts are used. The instructions should either pass -n <env> or point contributors to create_conda_env.py/update_conda_env.py so setup succeeds【F:docs/README.md†L24-L32】【F:devtools/conda-envs/docs_env.yaml†L1-L18】
Useful? React with 👍 / 👎.
Summary
.nbconvert.last_runmarkers and include a docs-focused pre-PR checklistTesting
https://chatgpt.com/codex/tasks/task_e_69082a46b5c48326b8aae92a69837211