refactor: move all color related code into separate file#4557
Merged
qwerty541 merged 1 commit intoanuraghazra:masterfrom Oct 11, 2025
Merged
Conversation
|
@qwerty541 is attempting to deploy a commit to the github readme stats Team on Vercel. A member of the Team first needs to authorize it. |
github-actions
bot
added
stats-card
qwerty541
approved these changes
Oct 11, 2025
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the codebase by extracting all color-related functionality from the main utils.js file into a dedicated color.js module for better code organization and separation of concerns.
Key changes:
- Created a new
src/common/color.jsfile containing all color utilities and theme functions - Removed color-related exports from
src/common/utils.jsandsrc/common/index.js - Updated import statements across test files and card modules to use the new color module
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/common/color.js | New module containing all color utilities: isValidHexColor, isValidGradient, fallbackColor, and getCardColors |
| src/common/utils.js | Removed color functions and updated JSDoc type reference for theme parameter |
| src/common/index.js | Removed color-related exports from main index |
| tests/utils.test.js | Updated import to use new color module for getCardColors |
| tests/card.test.js | Updated import to use new color module for getCardColors |
| src/cards/*.js | Updated imports across card modules to use new color module |
| scripts/preview-theme.js | Updated import to use new color module for validation functions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
qwerty541
deleted the
refactor_move_color_related_code_into_separate_file
branch
harryzcy
pushed a commit
to harryzcy/github-readme-stats
that referenced
this pull request
Oct 16, 2025
…#4557) Co-authored-by: Alexandr <qwerty541zxc@gmail.com>
martin-mfg
pushed a commit
to stats-organization/github-stats-extended
that referenced
this pull request
Nov 2, 2025
…#4557) Co-authored-by: Alexandr <qwerty541zxc@gmail.com>
ghosind
pushed a commit
to ghosind/github-readme-stats
that referenced
this pull request
Dec 30, 2025
…#4557) Co-authored-by: Alexandr <qwerty541zxc@gmail.com>
zimya
pushed a commit
to zimya/github-readme-stats-cf
that referenced
this pull request
Jan 13, 2026
…#4557) Co-authored-by: Alexandr <qwerty541zxc@gmail.com>
piscis
pushed a commit
to piscis/github-readme-stats
that referenced
this pull request
Feb 7, 2026
…#4557) Co-authored-by: Alexandr <qwerty541zxc@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.