Conversation
dceoy
force-pushed
the
docs/remove-pdmt5-api-docs
branch
from
553b227 to
1d5a8fb
Compare
![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: 1d5a8fb224
ℹ️ 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".
| Run the local QA script `scripts/qa.sh` in this skill. | ||
|
|
||
| ## Procedure | ||
|
|
||
| - Execute the script exactly as shown above when this skill is triggered. |
There was a problem hiding this comment.
The updated instructions no longer provide an executable command block and now reference scripts/qa.sh, which does not exist at the repository root; when this skill is triggered from the default repo working directory, following the text literally fails before QA starts (No such file or directory). This breaks the required QA workflow for any documentation/code change and can let unvalidated changes proceed.
Useful? React with 👍 / 👎.
1 participant
Summary
Remove the pdmt5-specific API documentation from the documentation site. The MT5 API now uses the unified documentation structure.
Changes
Test plan