Skip to content

Conversation

@caunt
Copy link
Owner

@caunt caunt commented Jan 20, 2026

Motivation

  • Fix a misspelling in a user-facing log message so runtime logs are consistent and clear.

Description

  • Updated the warning message in src/Plugins/Common/Services/Commands/AbstractCommandService.cs to use the spelling "canceled" instead of "cancelled".

Testing

  • Ran dotnet build (succeeded) and dotnet test which executed the test suite with results: 77 passed, 4 failed, 10 skipped; failing tests were ReadMinecraftBufferExtensionsTests.ReadTag_RoundTripsCompound, ReadMinecraftBufferExtensionsTests.ReadComponent_LegacyText_RoundTrips, and NbtReaderTests.ReadCompound_ReplacesWhitespaceTagNameWithText.

Codex Task

Behavior: warning message now uses "canceled" spelling.
Copilot AI review requested due to automatic review settings January 20, 2026 03:37
@caunt caunt merged commit 5612838 into main Jan 20, 2026
19 checks passed
@caunt caunt deleted the codex/fix-typos-in-code branch January 20, 2026 03:37
Copy link

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 corrects a spelling inconsistency in a user-facing log message, changing "cancelled" to "canceled" to align with American English conventions used elsewhere in user-facing strings.

Changes:

  • Updated warning log message in AbstractCommandService to use "canceled" instead of "cancelled"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants