Conversation
…tables() functions - disable_chronicle(conn, table_name): Remove chronicle tracking from a table, dropping the chronicle table, triggers, and indexes. Returns True if disabled, False if no chronicle existed. - is_chronicle_enabled(conn, table_name): Check if chronicle tracking is enabled for a table. - list_chronicled_tables(conn): Return list of all tables with chronicle tracking. - CLI --disable flag: python -m sqlite_chronicle db.sqlite table --disable Includes comprehensive tests and README documentation for all new functionality. Closes #6.
The chronicle table name in the FROM clause was not quoted, which would cause SQL errors for tables with special characters (spaces, dots, etc.) in their names. Added test to verify updates_since() works with special table names.
Add more detailed documentation explaining: - What triggers are created by _chronicle_triggers - Parameters and return values for updates_since - Purpose and return codes for cli_main
- Add mypy to test dependencies in pyproject.toml - Improve type hints: use Tuple[Any, ...] and Dict[str, Any] for Change dataclass - Add type annotation for cli_main argv parameter - Add type: ignore comment for sqlite3.Row row_factory assignment - Fix typo in README: ON CONFLICT SET -> ON CONFLICT DO UPDATE All type hints now pass mypy --strict. Note: GitHub Actions workflow update for mypy pending (requires workflows permission).
Resolve conflicts and fix disable_chronicle/list_chronicled_tables to handle the snapshot table introduced in main. Co-Authored-By: Claude Opus 4.6 <[email protected]>
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
2 participants
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.
disable_chronicle(conn, table_name): Remove chronicle tracking from a table, dropping the chronicle table, triggers, and indexes. Returns True if disabled, False if no chronicle existed.is_chronicle_enabled(conn, table_name): Check if chronicle tracking is enabled for a table.list_chronicled_tables(conn): Return list of all tables with chronicle tracking.python -m sqlite_chronicle db.sqlite table --disableIncludes comprehensive tests and README documentation for all new functionality.
Closes #6.