liamjd/UntitledColonyBuilder
1
0
Fork 0
Code Issues 36 Pull requests Projects 1 Releases Packages Wiki Activity Actions

Drop stale pawn references from highlight test comments #157

Merged
liamjd merged 1 commit from 143-highlight-test-comments into main 2026-10-11 20:08:09 +01:00
Conversation 0 Commits 1 Files changed 1 +11 -9
liamjd commented 2026-10-11 19:50:29 +01:00
Owner
Copy link

This is a follow-up to #143 (PR #155) and changes comments only.

test_selection_highlight.gd still described the overlay as something pawns wear, and justified the overlay slot partly by PawnView.flash(). Since #143 pawns use a ring instead, so the header, a helper's doc comment and one test's doc comment now talk about buildings only, and the header points to test_pawn_selection_ring.gd for the ring tests.

There are no code changes. test_selection_highlight.gd passes 8/8.

The other loose end from #143, the ring clipping into terrace risers, is filed as #156.

🤖 Generated with Claude Code

This is a follow-up to #143 (PR #155) and changes comments only. `test_selection_highlight.gd` still described the overlay as something pawns wear, and justified the overlay slot partly by `PawnView.flash()`. Since #143 pawns use a ring instead, so the header, a helper's doc comment and one test's doc comment now talk about buildings only, and the header points to `test_pawn_selection_ring.gd` for the ring tests. There are no code changes. `test_selection_highlight.gd` passes 8/8. The other loose end from #143, the ring clipping into terrace risers, is filed as #156. 🤖 Generated with [Claude Code](https://claude.com/claude-code) 
Pawns stand in a ring since #143, so the overlay tests are about
buildings only.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
liamjd merged commit 92253f812e into main 2026-10-11 20:08:09 +01:00
liamjd deleted branch 143-highlight-test-comments 2026-10-11 20:08:09 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
assets
Art, 3D models, UI themes, sound, music

  
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
testing
Unit and manual test activities

  
ui
User Interface

  
wontfix
This won't be fixed

No labels
assets
bug
duplicate
enhancement
help wanted
invalid
question
testing
ui
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
liamjd/UntitledColonyBuilder!157
No description provided.