Skip to content

Implement and document AnalysisConfig, add milestone tracking, remove web bundles, and change script permissions - #23

Merged
hyperpolymath merged 2 commits into
mainfrom
coderabbit/changes/a81cdab0
Sep 18, 2026
Merged

hyperpolymath merged 2 commits into
mainfrom
coderabbit/changes/a81cdab0

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

The eight commits go substantially beyond the requested PR #22 docstrings:

  • Add the canonical immutable AnalysisConfig implementation with compatibility aliases, stronger validation, risk banners, provenance, and DOI bundles.
  • Update JSON/Nickel schemas, the DEED template, frontend types and helpers, API documentation, and Julia unit tests.
  • Add Milestone 3 documentation, six deferred issue drafts, project-board automation, and tracking exceptions in .gitignore.
  • Remove committed web/dist artifacts and executable permissions from the commit hook, four utility scripts, and start.sh.

Validation was not run for this metadata task. The committed milestone report records 551 frontend tests passing, 11 failing, 5 todo, and 2 errors; Julia project tests timed out during precompilation, while isolated smoke checks reportedly passed.

View coding task

@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Author

Review Change StackReview Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Essentials

Run ID: 60fc23e6-f04f-4700-a5d1-2550d079b539

📥 Commits

Reviewing files that changed from the base of the PR and between defe967 and d4a1bbe.

📒 Files selected for processing (2)
  • frontend/src/types/analysis_config.ts
  • src/analysis/AnalysisConfig.jl
 ________________________________
< NVIDIA inside, Rabbit outside. >
 --------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath merged commit 4ea9ed0 into main Sep 18, 2026
@hyperpolymath
hyperpolymath deleted the coderabbit/changes/a81cdab0 branch September 18, 2026 23:55
@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant