-
-
Notifications
You must be signed in to change notification settings - Fork 6
fix: add support for alpha() color function
#107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The |
nzakas
left a comment
There was a problem hiding this 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.
|
Can you double-check the lint failures? |
Pull Request Test Coverage Report for Build 21523542999Warning: 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
💛 - Coveralls |
|
Thank you for reviewing and approving! |
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 thealpha()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!