Skip to content

Added @Override to execute() in 10 command files via parallel edits, then checked for test targets (none found) and verified the diffs.#470

Closed
opencode-agent[bot] wants to merge 1 commit into
masterfrom
opencode/issue466-20260606092247
Closed

Added @Override to execute() in 10 command files via parallel edits, then checked for test targets (none found) and verified the diffs.#470
opencode-agent[bot] wants to merge 1 commit into
masterfrom
opencode/issue466-20260606092247

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Jun 6, 2026

Copy link
Copy Markdown

All 10 files updated. The cli subproject has no test target, so ant test is N/A — the change is purely additive (@Override, java.lang, no imports needed, no behavioral change).

Summary

Added missing @Override annotation to execute() in 10 command classes across cli/src/commands/.

Changes

  • cli/src/commands/org/jnode/command/system/NamespaceCommand.java:47 — +@Override
  • cli/src/commands/org/jnode/command/system/MemoryCommand.java:50 — +@Override
  • cli/src/commands/org/jnode/command/system/CpuIDCommand.java:46 — +@Override
  • cli/src/commands/org/jnode/command/common/ExitCommand.java:46 — +@Override
  • cli/src/commands/org/jnode/command/common/TrueCommand.java:45 — +@Override
  • cli/src/commands/org/jnode/command/common/FalseCommand.java:45 — +@Override
  • cli/src/commands/org/jnode/command/common/EchoCommand.java:55 — +@Override
  • cli/src/commands/org/jnode/command/common/UptimeCommand.java:47 — +@Override
  • cli/src/commands/org/jnode/command/common/SleepCommand.java:51 — +@Override
  • cli/src/commands/org/jnode/command/file/PwdCommand.java:42 — +@Override

Closes #466

New%20session%20-%202026-06-06T09%3A22%3A46.612Z
opencode session  |  github run

…ts, then checked for test targets (none found) and verified the diffs.

Co-authored-by: LSantha <LSantha@users.noreply.github.com>
@LSantha

LSantha commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Closing as duplicate of #471. Both PRs make byte-identical changes (same 10 file SHAs) for the @OverRide sweep on #466. PR #471 is preferred because it is based on the latest master (91d385c, includes the orchestrator + skill fix) and reports as MERGEABLE/CLEAN, whereas #470 is based on 5012ed8 and is now behind master. All work for #466 will be merged via #471.

@LSantha LSantha closed this Jun 6, 2026
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.

Add @Override annotation to execute() in 10 small command classes

1 participant