Skip to content

Move candidates dir to repo root - #32

Merged
justincasher merged 1 commit into
mainfrom
justin/move-candidates-to-top-level
May 10, 2026
Merged

justincasher merged 1 commit into
mainfrom
justin/move-candidates-to-top-level

Conversation

@justincasher

Copy link
Copy Markdown
Collaborator

Summary

  • aggregator/ only ever held candidates/, so the wrapping directory was a redundant level. Move aggregator/candidates/ to candidates/ and update every reference (.gitignore, CANDIDATES_DIR in __main__.py, docstrings in manual.py/render.py/cloner.py).
  • The Python package keeps its lean_pool.aggregator name — that's the code that does the aggregation, not the data.
  • All 74 tests pass; python -m lean_pool.aggregator render writes to the new path correctly.

The wrapping aggregator/ directory only ever held candidates/ and
added a redundant level. Move aggregator/candidates/ to candidates/
and update every reference (gitignore, CANDIDATES_DIR in __main__,
docstrings in manual.py / render.py / cloner.py). The Python
package keeps its lean_pool.aggregator name since it's the code
that does the aggregation, not the data.
@justincasher
justincasher merged commit 3398274 into main May 10, 2026
2 of 3 checks passed
@Vilin97
Vilin97 deleted the justin/move-candidates-to-top-level branch May 11, 2026 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant