Skip to content

Conversation

Copy link

Copilot AI commented Dec 21, 2025

Addresses question on PR #9 about helper function refactoring to reduce code duplication in response colour coding.

Changes

  • Added getColourString() helper function to encapsulate RGB string formatting
  • Extracted repeated color string construction (8 occurrences) into single variable

Copilot AI changed the title [WIP] Add helper functions for response color coding Refactor RGB color string construction into helper function Dec 21, 2025
Copilot AI requested a review from JoshdRod December 21, 2025 15:15
@JoshdRod JoshdRod marked this pull request as ready for review December 21, 2025 15:16
@JoshdRod JoshdRod force-pushed the copilot/sub-pr-9-again branch from c525e93 to 113785f Compare December 21, 2025 19:39
@JoshdRod JoshdRod changed the title Refactor RGB color string construction into helper function <copilot><refactor> Move correctness -> rgb string into helper function Dec 21, 2025
@JoshdRod JoshdRod changed the title <copilot><refactor> Move correctness -> rgb string into helper function <refactor><copilot> Move correctness -> rgb string into helper function Dec 21, 2025
@JoshdRod JoshdRod force-pushed the copilot/sub-pr-9-again branch from 113785f to 5e532ad Compare December 21, 2025 19:41
@JoshdRod JoshdRod force-pushed the copilot/sub-pr-9-again branch from 5e532ad to 7d71c94 Compare December 21, 2025 19:43
@JoshdRod JoshdRod merged commit f75ecb3 into add-correctness-evaluator Dec 21, 2025
@JoshdRod JoshdRod deleted the copilot/sub-pr-9-again branch January 4, 2026 21:38
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