Skip to content

[cleanup] Remove unused validation scaffolding - #32

Merged
gauglertodd merged 2 commits into
mainfrom
cleanup/remove-unused-code
Sep 2, 2026
Merged

[cleanup] Remove unused validation scaffolding#32
gauglertodd merged 2 commits into
mainfrom
cleanup/remove-unused-code

Conversation

@gauglertodd

@gauglertodd gauglertodd commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove an orphaned comparison validator and unused type alias
  • remove a never-raised exception and unused status enum
  • align abstract return annotations with the implementations
  • preserve all tracing decorators and the tracing extension point unchanged
  • preserve exported compatibility models and merge-mode aliases

Validation

  • make format && make lint
  • python -m pytest -q gammalake/tests — 142 passed

Delete orphaned validation types, stale status annotations, and behaviorless tracing wrappers while preserving exported compatibility models.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Signed-off-by: Todd Gaugler <16946312+gauglertodd@users.noreply.github.com>
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Test Results

142 tests  ±0   142 ✅ ±0   45s ⏱️ -4s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit a252b76. ± Comparison against base commit f756fd2.

♻️ This comment has been updated with latest results.

Keep the tracing compatibility layer available for a future concrete tracing implementation.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Signed-off-by: Todd Gaugler <16946312+gauglertodd@users.noreply.github.com>
@gauglertodd gauglertodd changed the title [cleanup] Remove unused compatibility scaffolding [cleanup] Remove unused validation scaffolding Sep 2, 2026
@gauglertodd
gauglertodd merged commit 2565e6a into main Sep 2, 2026
3 checks passed
@gauglertodd
gauglertodd deleted the cleanup/remove-unused-code branch September 2, 2026 17:15
gauglertodd added a commit that referenced this pull request Sep 3, 2026
* [cleanup] Remove unused compatibility scaffolding

Delete orphaned validation types, stale status annotations, and behaviorless tracing wrappers while preserving exported compatibility models.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Signed-off-by: Todd Gaugler <16946312+gauglertodd@users.noreply.github.com>

* [cleanup] Preserve tracing extension point

Keep the tracing compatibility layer available for a future concrete tracing implementation.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Signed-off-by: Todd Gaugler <16946312+gauglertodd@users.noreply.github.com>

---------

Signed-off-by: Todd Gaugler <16946312+gauglertodd@users.noreply.github.com>
Co-authored-by: Todd Gaugler <16946312+gauglertodd@users.noreply.github.com>
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