Move packages into a top-level workspace - #339
Merged
Merged
Conversation
asinghvi17
requested review from
felixcremer,
lazarusA,
meggart,
mkitti and
nhz2
September 12, 2026 01:31
Coverage Report for CI Build 34696473871Coverage decreased (-4.9%) to 90.488%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
Member
|
Overall looks very good to me, just 2 questions:
|
Member
Author
|
Good point, should just make them all 0.11.0 then |
This was referenced Sep 12, 2026
Member
well... not sure. All sub packages are new, so is not normal to start with such high version numbers, plus they will diverge over time, so after a while it will not matter. I'm just curious about what is the best practice in this case, I don't know, do we match the version of the main package or do we start from |
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
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.
Moves Zarr and its eight component packages into sibling directories at the repository root. The root Project.toml contains only workspace membership; every local package dependency has a relative source path. Documentation stays in docs/, while the coverage and logo tooling environments remain outside the shared workspace.
Updates CI, documentation builds, coverage paths, CompatHelper, and TagBot for the new layout. Each package gets a standardized short README and a LICENSE.md symlink to the root license. All nine packages are version 0.11.0, with matching sibling dependency compatibility bounds.
The first commit contains only file moves: Git identifies all 71 as 100% identical renames. Configuration changes and documentation follow in separate commits.
Validation: the workspace resolves from a fresh manifest on Julia 1.13; all 31 focused package-location and registration checks pass. All relative source paths and license links were checked, workflow YAML parses, and git diff --check passes. Full CI and the documentation build have not been run locally.