test(isRgbColor): fix test expectations by WikiRik · Pull Request #2538 · validatorjs/validator.js

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@WikiRik

Copy link Copy Markdown

Member

@WikiRik WikiRik commented

Mar 27, 2025

Fixes the wrong test expectations for isRgbColor that occurred due to a former merge conflict. Also updates typo in package.json

Checklist

  • PR contains only changes related; no stray files, etc.
  • README updated (where applicable)
  • Tests written (where applicable)
  • References provided in PR (where applicable)
Also updates typo in package.json

@WikiRik

Copy link Copy Markdown

Member Author

WikiRik commented

Mar 27, 2025

@rubiin @profnandaa sorry for the ping, but would be great if we could merge this PR soon so we can have tests passing on master again

@WikiRik WikiRik mentioned this pull request

Mar 28, 2025

4 tasks

@WikiRik

Copy link Copy Markdown

Member Author

WikiRik commented

Apr 8, 2025

@rubiin @profnandaa sorry for the ping, but would be great if we could merge this PR soon so we can have tests passing on master again

@rubiin @profnandaa reminder for the above, tests are failing on master at this moment

@profnandaa profnandaa merged commit fc7a60a into master

Apr 8, 2025

10 checks passed

@WikiRik WikiRik deleted the WikiRik/isRgbColor-tests branch

April 8, 2025 15:04

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

Reviewers

@profnandaa profnandaa profnandaa approved these changes

@rubiin rubiin Awaiting requested review from rubiin

+1 more reviewer

@vitorgamer58 vitorgamer58 vitorgamer58 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@WikiRik @profnandaa @vitorgamer58 @rubiin