Skip to content

Add support for tax liability adjustments - #66

Merged
ac000 merged 1 commit into
masterfrom
tla
Oct 12, 2026
Merged

ac000 merged 1 commit into
masterfrom
tla

Conversation

@ac000

@ac000 ac000 commented Oct 12, 2026

Copy link
Copy Markdown
Owner

This allows to view, amend & delete individual tax liability adjustments.

Copilot AI balanced review requested due to automatic review settings October 12, 2026 16:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The functionality is coherent, with only a localized coding-style issue remaining.

1 open finding
What changed in this PR

Adds CLI support for viewing, amending, and deleting individual tax liability adjustments.

Changes:

  • Adds three tax-liability adjustment commands and handlers.
  • Documents the new functionality and command syntax.
File Description
src/​itsa.c Implements and dispatches the new commands.
README.rst Documents the supported actions and commands.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/itsa.c Outdated
This allows to view, amend & delete individual tax liability
adjustments.

Signed-off-by: Andrew Clayton <ac@sigsegv.uk>
Copilot AI balanced review requested due to automatic review settings October 12, 2026 16:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The destructive tax API operations lack automated functional coverage and require human validation against the external API contract.

0 open findings

1 resolved since last review

🧠 Review effort: Balanced

@ac000
ac000 merged commit 7c86a14 into master Oct 12, 2026
8 checks passed
@ac000
ac000 deleted the tla branch October 12, 2026 16:35
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