chore: clean up top-level + add LICENSE (registration prep)#18
Merged
Conversation
Removes accumulated detritus from earlier dev sessions and prepares
the package for General Registry submission.
Deleted:
- One-off codegen scripts whose output is already in src/:
refactor.py, refactor_complete.jl, refactor_read_record.jl,
add_instdef_helpers.py, add_missing_helpers.jl
- Historical performance / optimization markdown reports
(info preserved in git history; live perf docs continue in
benchmark/{PERFORMANCE_REPORT,BENCHMARK_RESULTS,EXAMPLE_OUTPUT}.md):
FUTURE_OPTIMIZATIONS.md, OPTIMIZATION_SUMMARY.md,
PARALLELIZATION_ANALYSIS.md, PERFORMANCE_OPTIMIZATION_REPORT.md,
PERFORMANCE_REPORT.md, STREAMING_OPTIMIZATION_SUMMARY.md
- Backup file: PERFORMANCE_REPORT.md.bak
Moved:
- CLAUDE.md -> .claude/CLAUDE.md (gitignored; project-local
Claude Code instructions, not part of the public package)
Added:
- LICENSE (MIT, matching DatabentoAPI.jl) — required by the
General Registry
No source or test changes; 3643/3643 tests still pass.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Comments on pull requests will not trigger Registrator, as it is disabled. Please try commenting on a commit or issue. |
BenchmarkTools is only used in test/test_phase10_complete.jl and benchmark/*.jl scripts — never in src/. Keeping it as a runtime dep would unnecessarily bloat the install for downstream users once DBN.jl is registered. It remains available to the test environment via [extras] + [targets].test = [..., "BenchmarkTools"], which is the standard pattern for test-only deps. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
2 participants
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.
Repo hygiene pass ahead of submitting DBN.jl to the General Registry.
Deleted
One-off codegen scripts (output already in
src/):refactor.py,refactor_complete.jl,refactor_read_record.jladd_instdef_helpers.py,add_missing_helpers.jlHistorical perf / optimization markdown reports. Info preserved in git history; live perf docs continue to live in
benchmark/{PERFORMANCE_REPORT,BENCHMARK_RESULTS,EXAMPLE_OUTPUT}.md:FUTURE_OPTIMIZATIONS.md,OPTIMIZATION_SUMMARY.md,PARALLELIZATION_ANALYSIS.md,PERFORMANCE_OPTIMIZATION_REPORT.md,PERFORMANCE_REPORT.md,STREAMING_OPTIMIZATION_SUMMARY.mdBackup file:
PERFORMANCE_REPORT.md.bakMoved
CLAUDE.md→.claude/CLAUDE.md(gitignored). Project-local Claude Code instructions; not part of the public package.Added
LICENSE(MIT, matching DatabentoAPI.jl). Required by the General Registry.Verification
src/,test/,docs/,benchmark/, orREADME.md(grepclean).README.md,LICENSE,Project.toml,Manifest.toml(gitignored), plussrc/,test/,docs/,benchmark/,.github/, etc.Next steps after this lands:
@JuliaRegistrator registerto submit DBN.jl v0.1.0.🤖 Generated with Claude Code