Skip to content

Repository folder structure#563

Draft
iameneko wants to merge 11 commits intodevelopfrom
560-repository-folder-structure
Draft

Repository folder structure#563
iameneko wants to merge 11 commits intodevelopfrom
560-repository-folder-structure

Conversation

@iameneko
Copy link
Copy Markdown
Contributor

@iameneko iameneko commented Apr 10, 2026

During this PR the following tasks have been completed:
The following tasks need to be completed to reach the target structure above.

  • Rename docs-website/ → docs/ and update the deploy-website.yml workflow reference.
  • Move unique files from doc/images/ into docs/static/ (skip duplicates already present in docs/static/img/ and docs/static/logos/).
  • Delete doc/.
  • Fix typo CI/environment/ → CI/environment/ and update any scripts referencing the old path.
  • Move .github/docker/dockerfile → CI/docker/, update the workflow that references it, and delete the now-empty .github/docker/ folder.
  • Delete .travis.yml (superseded by GitHub Actions).
  • Delete ChronoStore/ (unused).
  • Create conf/ and move default_conf.json.in and default_client_conf.json.in into it. Update CMakeLists.txt references.
  • Move test_suite.md → tests/.
  • Move CodeStyleFiles/ → .github/code-style/.
  • Rename plugin subdirectories: chronokvs/ → chrono-kvs/, chronostream/ → chrono-stream/, chronoviz/ → chrono-viz/.
  • Create src/ and move ChronoVisor/, ChronoKeeper/, ChronoGrapher/, ChronoPlayer/, chrono_common/ into it, renaming them to chrono-visor/, chrono-keeper/, chrono-grapher/, chrono-player/, chrono-common/. Update all CMakeLists.txt and workflow references.
  • Rename Client/ → client/ and Plugins/ → plugins/. Update all CMakeLists.txt references.
  • Rename CI/ → ci/. Update all workflow references.

iameneko and others added 10 commits April 10, 2026 20:47
Travis CI is fully superseded by GitHub Actions workflows.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Rename docs-website/ -> docs/ (Docusaurus site)
- Merge unique images from doc/images/ into docs/static/img/ and docs/static/logos/
- Delete legacy doc/ directory
- Update deploy-website.yml path triggers and working-directory references
- Update CMakeLists.txt Doxygen and lint --exclude paths from doc/ to docs/

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Move .github/docker/dockerfile -> ci/docker/ (alongside existing Dockerfiles)
- Delete now-empty .github/docker/
- Rename CI/ -> ci/ for consistent lowercase directory naming
- Update all workflow docker context and hashFiles() references from
  .github/docker/** to ci/docker/**

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Create conf/ directory
- Move default_conf.json.in and default_client_conf.json.in from root into conf/
- Update all configure_file() and install(FILES ...) calls in CMakeLists.txt:
  src/chrono-visor (formerly ChronoVisor), client/cpp, tools/cli,
  tests/integration/keeper-grapher

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Rename CodeStyleFiles/ -> .github/code-style/ to keep code style config
  co-located with GitHub Actions and tooling configuration
- Update check-format.yml: trigger path, clang-format style arg,
  PR comment instructions, and header comment
- Update pre-commit hook: fix internal path reference to clang-format config

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Align with target directory layout (tests/ plural).
Update add_subdirectory(test) -> add_subdirectory(tests) in root CMakeLists.txt.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Rename Client/ -> client/ for consistent lowercase naming
- Update root CMakeLists.txt: add_subdirectory(Client) -> add_subdirectory(client)
- Update hashFiles() in integration-pipeline, publish-dev-image, create-release
  workflows: Client/spack.yaml -> client/spack.yaml
- Update all CMAKE_SOURCE_DIR and CMAKE_BINARY_DIR Client/ path references
  across component, plugin, test, and tools CMakeLists.txt files

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Rename Plugins/ -> plugins/
- Rename chronokvs/ -> chrono-kvs/, chronostream/ -> chrono-stream/,
  chronoviz/ -> chrono-viz/ for consistency with kebab-case component naming
- Update root CMakeLists.txt: add_subdirectory(Plugins) -> add_subdirectory(plugins)
- Update plugins/CMakeLists.txt subdirectory names
- Update test include paths for chrono-kvs headers in unit and integration tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Create src/ directory for all server-side ChronoLog components
- Move and rename:
    ChronoVisor/    -> src/chrono-visor/
    ChronoKeeper/   -> src/chrono-keeper/
    ChronoGrapher/  -> src/chrono-grapher/
    ChronoPlayer/   -> src/chrono-player/
    chrono_common/  -> src/chrono-common/
- Update root CMakeLists.txt add_subdirectory() calls for all five components
- Update all CMAKE_SOURCE_DIR path references across component, client,
  plugin, and test CMakeLists.txt files:
    chrono_common/ -> src/chrono-common/
    ChronoVisor/   -> src/chrono-visor/
    ChronoKeeper/  -> src/chrono-keeper/
    ChronoGrapher/ -> src/chrono-grapher/
    ChronoPlayer/  -> src/chrono-player/
- Fix stale path comments in src/chrono-common and src/chrono-player

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Cleans up the remaining tracked-but-deleted paths left over from
the CI/ -> ci/ rename in the previous commit.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@iameneko iameneko self-assigned this Apr 10, 2026
@github-actions

This comment was marked as resolved.

…sistency and readability

- Adjusted formatting for constructor and method signatures in ChronologClient.cpp for improved readability.
- Consolidated destructor and method implementations into single-line definitions where appropriate.
- Updated header files in chrono-common and plugins to standardize spacing and formatting, enhancing code clarity.
- Ensured consistent use of references and const qualifiers across method signatures.

This refactor aims to enhance code maintainability and readability without altering functionality.
@iameneko iameneko marked this pull request as ready for review April 13, 2026 16:03
@iameneko iameneko requested review from fkengun and ibrodkin April 13, 2026 16:03
@iameneko iameneko linked an issue Apr 13, 2026 that may be closed by this pull request
@iameneko iameneko added the Maintenance Refactoring, cleanup, dependency updates, dead-code removal label Apr 14, 2026
@iameneko iameneko marked this pull request as draft April 15, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance Refactoring, cleanup, dependency updates, dead-code removal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Repository Folder Structure

1 participant