Skip to content

Make Stone easier to invoke locally#363

Merged
KevinRK29 merged 1 commit into
dropbox:mainfrom
aaronsky:codex/asky/stone-local-invocation
May 6, 2026
Merged

Make Stone easier to invoke locally#363
KevinRK29 merged 1 commit into
dropbox:mainfrom
aaronsky:codex/asky/stone-local-invocation

Conversation

@aaronsky
Copy link
Copy Markdown
Contributor

@aaronsky aaronsky commented May 6, 2026

Description

Stacked on #359.

Allows stone/cli.py to bootstrap its package path when run directly and adds stone/__main__.py for module invocation.

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-local-invocation branch from d1e083e to d99b67f Compare May 6, 2026 13:48
@aaronsky aaronsky force-pushed the codex/asky/stone-recursive-specs branch from 91354a0 to 1786e6b Compare May 6, 2026 13:48
@aaronsky aaronsky force-pushed the codex/asky/stone-local-invocation branch from d99b67f to 2d433c3 Compare May 6, 2026 14:00
@aaronsky aaronsky force-pushed the codex/asky/stone-recursive-specs branch from 1786e6b to 9f03669 Compare May 6, 2026 14:00
@aaronsky aaronsky changed the base branch from codex/asky/stone-recursive-specs to codex/asky/stone-output-manifest May 6, 2026 19:03
@aaronsky aaronsky force-pushed the codex/asky/stone-output-manifest branch from 9f03669 to fb99efd Compare May 6, 2026 20:19
@aaronsky aaronsky force-pushed the codex/asky/stone-local-invocation branch from 2d433c3 to 01e9971 Compare May 6, 2026 20:23
@aaronsky aaronsky changed the base branch from codex/asky/stone-output-manifest to main May 6, 2026 20:24
@KevinRK29 KevinRK29 merged commit 7a087d2 into dropbox:main May 6, 2026
8 checks passed
@aaronsky aaronsky requested a review from KevinRK29 May 6, 2026 20:26
@aaronsky aaronsky deleted the codex/asky/stone-local-invocation branch May 6, 2026 20:26
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.30%. Comparing base (1ad079b) to head (01e9971).
⚠️ Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
stone/__main__.py 0.00% 3 Missing ⚠️
stone/cli.py 0.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #363      +/-   ##
==========================================
+ Coverage   49.98%   51.30%   +1.31%     
==========================================
  Files          37       38       +1     
  Lines        8808     8876      +68     
  Branches     1892     1913      +21     
==========================================
+ Hits         4403     4554     +151     
+ Misses       4087     3979     -108     
- Partials      318      343      +25     
Flag Coverage Δ
unit 51.30% <0.00%> (+1.31%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

4 participants