Skip to content

Conversation

@rogerpearce
Copy link
Collaborator

@rogerpearce rogerpearce commented Dec 18, 2025

Lets use this checklist to track things needed before merging

  • Move python code into py directory.
  • Copy clippy-cpp/master into cpp directory. Remove old and dead code
  • Fix old examples that have been moved into examples/SimpleExamples. Currently not connected to build system.
  • Move tests into examples and rename, e.g. TestBag -> ExampleBag
  • Update pytests to use ExampleBag, ExampleSet, etc.
  • Integrated pytest at top level of repo? (Decision via msg: in test/ dir at top of repo)
  • Add SimpleExamples testing to pytest
  • Update top level Readme with current instructions.

Questions?

  • Could we have a top level CmakePresets file that would allow developer to build easily from the top level?
  • Should we remove files in cpp/tests (they look redundant to top level test now). Later we could add in a cpp/tests if we create self-contained cpp-only tests.

@sbromberger
Copy link
Member

Ref: #31

@sbromberger
Copy link
Member

my thought: integrated pytest should be in a test directory at the top of the repo.

@sbromberger sbromberger mentioned this pull request Dec 19, 2025
@rogerpearce rogerpearce marked this pull request as ready for review December 19, 2025 19:36
@sbromberger sbromberger merged commit 8ac64b1 into master Dec 19, 2025
1 check passed
@sbromberger sbromberger deleted the monorepo branch December 19, 2025 20:12
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.

3 participants