chore: align .github structure with Piccolo conventions - #10
Merged
Conversation
Bring HBJ's `.github/` up to parity with the Piccolo.jl-shaped layout that the rest of the org uses. Files are copied verbatim from Piccolo unless noted. Workflows --------- - **CI.yml**: replace the precompile-only smoke check with the canonical matrix test workflow — runs `julia-runtest`, processes coverage, and uploads to codecov on Julia 1.10/1.11/1.12 + Linux x64. Adds `paths-ignore: ['docs/**', '*.md']` so doc-only PRs don't burn CI minutes once docs/ exists. - **Formatter.yml** *(new)*: JuliaFormatter check on PR/push, with workflow_dispatch escape hatch to autoformat + commit. - **nightly.yml** *(new)*: daily Julia 'pre' install check (matches Piccolo — install-only, not test run; intentionally lighter than DTO's nightly). - **TagBot.yml**: cosmetic — unquote `lookback: 3` to match Piccolo exactly (functionally identical, both parse as string). (Piccolo's `tasksmd-sync.yml` is intentionally omitted — HBJ doesn't use the TASKS.md convention and adding the workflow speculatively is clutter.) dependabot.yml -------------- Add the `julia` ecosystem alongside `github-actions` so dependabot opens compat-bump PRs against Project.toml weekly. Comments restored to Piccolo's verbose form. CI.yml previously only checked that the package precompiles and loads — so the test suite was never actually run on CI. This commit fixes that gap; downstream branches (e.g. the alloc-analyzer PR #8) will rebase on top of this to pick up the real test runs.
jack-champagne
force-pushed
the
chore/github-alignment
branch
from
May 20, 2026 06:40
98fc9d6 to
f5c08c4
Compare
The Formatter workflow added in the previous commit will reject any
source file that isn't JuliaFormatter-clean. HBJ has never had a
formatter check, so the existing tree had accumulated formatter debt
(multi-line arg lists kept on one line, short conjunctions split
across multiple lines, etc).
Apply `JuliaFormatter.format(".")` over the whole repo so the Formatter
CI check on this PR (and every PR thereafter) starts from a clean
baseline. No semantic changes.
Touched: examples/convergence_template.jl, src/extractors.jl,
src/harness.jl, src/report.jl, src/schema.jl, src/storage.jl,
test/runtests.jl.
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.
Summary
Brings HBJ's
.github/up to parity with the Piccolo.jl-shaped layout therest of the org uses. Will land before HBJ#8 (alloc analyzer), which is being
rebased on top of this so it picks up the real test runs.
What changes
Workflows
CI.ymljulia-runtest+ coverage + codecov on Julia 1.10/1.11/1.12 × Linux x64. Addspaths-ignore: ['docs/**', '*.md']so doc-only PRs don't burn minutes once docs/ exists.Formatter.yml(new)workflow_dispatchescape hatch to autoformat + commit.nightly.yml(new)julia-runtest; we follow Piccolo here).TagBot.ymllookback: 3to match Piccolo exactly. Functionally identical.Dependabot
dependabot.yml: add thejuliaecosystem alongsidegithub-actionsso weekly compat-bump PRs land againstProject.tomltoo. Restores Piccolo's verbose comments.Why now
The previous
CI.ymlonly ranusing HarmoniqsBenchmarks— i.e. CI never actually ran the test suite. Locally the 295-test suite passes, but CI gave a misleading green. This PR closes that gap by switching tojulia-runtest.Test plan
nightly.ymldoesn't run on PRs,Files copied verbatim from
harmoniqs/Piccolo.jl/.github/All five workflows +
dependabot.ymlare byte-for-byte from Piccolo (sole exception:TagBot.ymlalready existed here, this PR just realigns the one quote-style nit).