Skip to content

Conversation

@rururux
Copy link

@rururux rururux commented Jan 25, 2026

Prerequisites checklist

What is the purpose of this pull request?

Added support for the alpha() function, which was the last remaining item from issue #82.

What changes did you make? (Give an overview)

In addition to setting the type definition for the alpha() function in patch.json, I also updated the color-function definition because the alpha() function was missing from the color-function definition in mdn-data/css.
I also added tests.

Related Issues

fixes #82

Is there anything you'd like reviewers to focus on?

This is my first time contributing to the ESLint community, so please feel free to point out anything missing.
Thank you in advance!

@eslint-github-bot eslint-github-bot bot added the bug Something isn't working label Jan 25, 2026
@eslint-github-bot eslint-github-bot bot mentioned this pull request Jan 25, 2026
8 tasks
@eslintbot eslintbot added this to Triage Jan 25, 2026
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Jan 25, 2026
@xbinaryx
Copy link

The alpha() color function was only recently added to the spec, and it’s not yet documented on MDN or implemented in any browser. Given the lack of real-world support, I don’t think we should add this yet.

@nzakas nzakas requested a review from SwetaTanwar January 30, 2026 16:39
nzakas
nzakas previously approved these changes Jan 30, 2026
Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really good, thank you!

Would like @SwetaTanwar to review before merging.

@nzakas
Copy link
Member

nzakas commented Jan 30, 2026

Can you double-check the lint failures?

@nzakas nzakas moved this from Needs Triage to Second Review Needed in Triage Jan 30, 2026
@coveralls
Copy link

coveralls commented Jan 30, 2026

Pull Request Test Coverage Report for Build 21523542999

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 98.557%

Totals Coverage Status
Change from base Build 20986935233: 0.006%
Covered Lines: 11968
Relevant Lines: 12109

💛 - Coveralls

@rururux
Copy link
Author

rururux commented Jan 30, 2026

Thank you for reviewing and approving! ☺️
I've fixed the lint issues.

@rururux rururux requested a review from nzakas January 30, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working contributor pool

Projects

Status: Second Review Needed

Development

Successfully merging this pull request may close these issues.

Fix relative color patch

4 participants