577 add comprehensive instructions for khiops python development#580
Open
popescu-v wants to merge 2 commits into
Open
577 add comprehensive instructions for khiops python development#580popescu-v wants to merge 2 commits into
popescu-v wants to merge 2 commits into
Conversation
- 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
tramora
reviewed
May 22, 2026
| (`.khj`, `.khcj`) | ||
| - `core.internals.runner` — backend abstraction for local, Docker, and other | ||
| execution modes, configurable with `get_runner()` and `set_runner()` | ||
| - `core.internals.filesystems` — filesystem abstraction for local, S3, and |
Collaborator
Author
There was a problem hiding this comment.
Yes, will update this.
tramora
requested changes
May 22, 2026
Collaborator
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
Collaborator
Author
Right, we could abstract these away. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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