fix(plotly): correct broken internal links reference/ → references/#117
fix(plotly): correct broken internal links reference/ → references/#117robotlearning123 wants to merge 2 commits into
Conversation
… references/ All 11 internal markdown links in neuropixels-analysis/SKILL.md pointed to `reference/` (without 's') but the actual directory is `references/`. This made every reference document link return 404. Changes: - 3 section headings: reference/ → references/ - 8 markdown links: reference/ → references/ Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
All 10 internal markdown links in plotly/SKILL.md pointed to `reference/` but the actual directory is `references/`. Changes: 10 link paths corrected (reference/ → references/) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7dd4b65ac7
ℹ️ 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".
|
|
||
| - Official documentation: https://plotly.com/python/ | ||
| - API reference: https://plotly.com/python-api-reference/ | ||
| - API reference: https://plotly.com/python-api-references/ |
There was a problem hiding this comment.
This change updates the external docs link from python-api-reference to python-api-references, which points to a nonstandard/invalid Plotly docs path and breaks the “API reference” pointer for users following this skill guide. Because this section is specifically meant to send readers to the canonical API docs, the incorrect URL defeats the purpose of the resource list and should be reverted to the valid singular path.
Useful? React with 👍 / 👎.
2 participants
Summary
plotly/SKILL.mdreference/but the actual directory isreferences/Bug Details
Same issue as #116 but for the plotly skill. All internal links used
reference/(no 's') while the filesystem directory isreferences/(with 's').Affected links:
reference/plotly-express.md(referenced 2x)reference/graph-objects.md(referenced 2x)reference/chart-types.md(referenced 2x)reference/layouts-styling.md(referenced 2x)reference/export-interactivity.md(referenced 2x)Test Plan
references/directory🤖 Generated with Claude Code