Conversation
CONTRIBUTING.md and the PR template still described the old single-directory layout, from before regressor.py moved into experiment/methods/ (b5c5c96). back then CI copied the files over via scripts/copy_algorithm_files.sh, so contributors only touched algorithms/. that script is gone but the docs never got updated, and nothing catches the mistake: the build-and-test matrix comes from `ls algorithms/`, so a method added only under experiment/methods/ never gets a job and still goes all green (see #211). - CONTRIBUTING.md documents where files go and why it's split that way - PR template reorganized around the two directories - local_ci.sh now wraps the docker commands instead of the old conda flow - scripts/check_method_layout.py checks the two dirs agree; runs in CI as validate-layout and gates build-and-test - fixes algorithms/eql/metadata.yml (invalid yaml), a missing experiment/methods/xgboost/__init__.py, and an invalid escape sequence in gplearn's regressor.py
lacava
added a commit
that referenced
this pull request
Aug 13, 2026
#220 was squash-merged, so master carried a commit with the same content as dev's 4629556 but a different hash. that made master and dev diverge and turned the next promote into a phantom conflict in ci-docker.yml. dev already contains everything in master, so this records the merge without taking any content. master is an ancestor of dev again and the next promote is a fast-forward. going forward: squash contributor PRs into dev, but promote dev -> master with a merge or fast-forward, never a squash.
Merged
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
1 participant
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.
CONTRIBUTING.md and the PR template still described the old single-directory layout, from before regressor.py moved into experiment/methods/ (b5c5c96). back then CI copied the files over via scripts/copy_algorithm_files.sh, so contributors only touched algorithms/. that script is gone but the docs never got updated, and nothing catches the difference. the build-and-test matrix comes from
ls algorithms/, so a method added only under experiment/methods/ never gets a job and still goes all green (see #211).