Skip to content

test(batch): add push/delete and media upload tests; docs/CI updates#18

Merged
mrMaxwellTheCat merged 2 commits intomainfrom
feat/batch-push-tests-docs
Feb 8, 2026
Merged

test(batch): add push/delete and media upload tests; docs/CI updates#18
mrMaxwellTheCat merged 2 commits intomainfrom
feat/batch-push-tests-docs

Conversation

@mrMaxwellTheCat
Copy link
Copy Markdown
Owner

This PR adds tests for batch-build push/delete behavior (including media upload and push failure cases), updates docs to standardize on 'just' and documents batch CLI status, and normalizes exceptions exports. All tests and quality checks pass locally (just all).\n\nChanges: \n- Add tests: ests/test_cli_batch.py\n- Update docs: README.md, CONTRIBUTING.md, ROADMAP.md, docs/*\n- Fix: src/anki_yaml_tool/core/exceptions.py (docstring and all)\n\nAll checks: ruff, mypy, pytest (173 tests) pass and coverage is above threshold.\n

Copilot AI review requested due to automatic review settings February 7, 2026 23:59
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR strengthens the CLI’s batch workflow validation by adding tests around batch-build push/delete behavior (including media upload and push failures), and updates documentation to standardize contributor commands on just. It also tidies the exceptions module exports.

Changes:

  • Add batch-build tests covering --push, --delete-after, media upload, and push-failure retention behavior.
  • Standardize contributor/release docs to use just commands (e.g., just all, just test, just lint).
  • Normalize core.exceptions module docstring and exported symbols via __all__.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/test_cli_batch.py Adds coverage for batch-build push/delete/media behaviors using monkeypatched AnkiConnector methods.
src/anki_yaml_tool/core/exceptions.py Updates module docstring and defines __all__ for consistent exception exports.
docs/VERSIONING.md Updates release checklist commands from make to just.
docs/DOCUMENTATION_ORGANIZATION.md Updates pre-commit workflow guidance to use just all.
CONTRIBUTING.md Updates contributor workflow snippets to use just commands.
.vscode/tasks.json Removes VSCode task definitions from the repo.

Comment thread docs/DOCUMENTATION_ORGANIZATION.md Outdated
@mrMaxwellTheCat mrMaxwellTheCat merged commit 3b042fa into main Feb 8, 2026
13 checks passed
@mrMaxwellTheCat mrMaxwellTheCat deleted the feat/batch-push-tests-docs branch February 8, 2026 00:15
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.

2 participants