Skip to content

Validate generated outputs against a manifest#361

Merged
KevinRK29 merged 1 commit into
dropbox:codex/asky/stone-output-root-validationfrom
aaronsky:codex/asky/stone-expected-output-manifest
May 6, 2026
Merged

Validate generated outputs against a manifest#361
KevinRK29 merged 1 commit into
dropbox:codex/asky/stone-output-root-validationfrom
aaronsky:codex/asky/stone-expected-output-manifest

Conversation

@aaronsky
Copy link
Copy Markdown
Contributor

@aaronsky aaronsky commented May 6, 2026

Description

Stacked on #360.

Adds --expected-output-manifest so Stone can compare generated output paths against a checked manifest.

Checklist

General Contributing

  • Have you read the Code of Conduct and signed the CLA?

Is This a Code Change?

  • Non-code related change (markdown/git settings etc)
  • Code Change
  • Example/Test Code Change

Validation

  • Have you ran tox?
  • Do the tests pass?

Verification run directly with the CI lint/test steps:

  • uv run --no-project --with-requirements requirements.txt --with-requirements test/requirements.txt --with flake8==5.0.4 --with pylint flake8 setup.py example stone test
  • uv run --no-project --with-requirements requirements.txt --with-requirements test/requirements.txt --with flake8==5.0.4 --with pylint --with setuptools pylint --rcfile=.pylintrc setup.py example stone test
  • uv run --no-project --with-requirements requirements.txt --with-requirements test/requirements.txt --with pytest --with mock pytest
  • uv run --no-project --with-requirements requirements.txt --with-requirements test/requirements.txt --with enum34 --with mypy --with types-six ./mypy-run.sh

@aaronsky aaronsky force-pushed the codex/asky/stone-expected-output-manifest branch from 2e58424 to 998f045 Compare May 6, 2026 13:48
@aaronsky aaronsky force-pushed the codex/asky/stone-output-root-validation branch from 69f3959 to 55f2793 Compare May 6, 2026 13:48
@aaronsky aaronsky force-pushed the codex/asky/stone-expected-output-manifest branch from 998f045 to bcf7dc9 Compare May 6, 2026 14:00
@aaronsky aaronsky force-pushed the codex/asky/stone-output-root-validation branch from 55f2793 to c96192c Compare May 6, 2026 14:00
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ musicinmybrain
✅ aaronsky
❌ rhui-dbx
You have signed the CLA already but the status is still pending? Let us recheck it.

@KevinRK29 KevinRK29 merged commit dc8a861 into dropbox:codex/asky/stone-output-root-validation May 6, 2026
1 check was pending
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.

5 participants