577 add comprehensive instructions for khiops python development#580
Conversation
There was a problem hiding this comment.
For an human-being like me it seems very clear and comprehensive.
From the root copilot-instructions.md it is easy to follow and apply the corresponding sub instructions.
I'm wondering if it is required to give the workflow counts (7) and the current components versions that will require instructions changes later whenever they change
Right, we could abstract these away. |
02411ce to
6e4e0ce
Compare
On second thought:
|
tramora
left a comment
There was a problem hiding this comment.
The current status of the instructions is fine for me. There is a fear though about a little repetition regarding the Python code formatting and dependencies within the Python code but it is not blocking.
All these instructions will have to be refreshed soon for the very next Pull Requests
|
I was surprised that the github workflows run after the instructions update. It was not the case the first time. |
It's because of the first commit which removes an unnecessary file in the |
- generic instruction file: .github/copilot-instructions.md - instruction file specific to the CI workflows: .github/instructions/ci-workflows.instructions.md - instruction file specific to the documentation generation: .github/instructions/doc-changes.instructions.md - instruction file specific to the development Docker image maintenance: .github/instructions/docker-changes.instructions.md - instruction file specific to the maintenance of the Python code itself: .github/instructions/python-changes.instructions.md
Update Copilot instructions accordingly.
1d73ab5 to
f007510
Compare
closes #577
TODO Before Asking for a Review
main(ormain-v10)Make sure all CI workflows are greenThis is moot for this kind of updatesWhen adding a public feature/fix: Update theThis is irrelevant to dev choresUnreleasedsection ofCHANGELOG.md(no date)index.html