chore: remove stale test and typecheck configuration - #49
Merged
Conversation
Drop an unused pytest marker, a permanently skipped bodyless test whose replacement is live, and a mypy override for a module that moved packages.
Deploying shepherd-agents-docs with
|
| Latest commit: |
875c9ee
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://382e09c3.shepherd-agents-docs.pages.dev |
| Branch Preview URL: | https://chore-stale-test-config-clea.shepherd-agents-docs.pages.dev |
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
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
workspace_componentpytest marker registration.shepherd_core.export.atifmodule while preserving the activeshepherd_export.atifoverride.Motivation / Context
These entries survived earlier moves and behavior changes but no longer affect a live code or test path. Removing them keeps marker output, device coverage, and type-check configuration aligned with the current repository without changing runtime behavior.
This is a tailored public cleanup; internal-only ledger and documentation-script changes are intentionally excluded.
Delta Breakdown
workspace_componentis registered but is not applied to or selected by any public test.test_device_context_nested_raises_errorremains and passes.shepherd_core.export.atifmodule no longer exists. The liveshepherd_export.atifoverride remains unchanged.Entry Point
Root mypy configuration → dialect pytest marker configuration → runtime device test cleanup.
There is no runtime or public API entrypoint.
Review Guide
Reviewers should verify that each removed entry is genuinely unreferenced and that the surviving replacement is the intended contract. In particular, this PR does not remove the active export override and does not change the nested-device prohibition.
Validation
uv lock --checkandgit diff --checkpassed.test_device_context_nested_raises_error.make verifyimports all workspace packages successfully.Documentation / Changelog / Decisions
No changelog or documentation update is needed because this removes inert configuration and a dead skipped test without user-visible behavior.
Risk / Rollback / Migration
Risk is near-zero: all removals were inert or redundant. Rollback is a normal revert. No migration or compatibility action is required.
Follow-ups
None.