Skip to content

Conversation

akulsr0
Copy link

@akulsr0 akulsr0 commented Aug 3, 2025

Prerequisites checklist

What is the purpose of this pull request?

Fixes #223

What changes did you make? (Give an overview)

Handle the cases when var is used inside calc fn

Related Issues

Fixes #223

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

@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Aug 3, 2025
@eslint-github-bot
Copy link

Hi @akulsr0!, thanks for the Pull Request

The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.

  • The commit message tag wasn't recognized. Did you mean "docs", "fix", or "feat"?
  • There should be a space following the initial tag and colon, for example 'feat: Message'.
  • The length of the commit message must be less than or equal to 72

To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page.

Read more about contributing to ESLint here

@akulsr0 akulsr0 changed the title fix(no-invalid-properties): incorrect report when var is used inside calc fn fix: no-invalid-properties incorrect report when var is used inside calc fn Aug 3, 2025
@eslint-github-bot
Copy link

Hi @akulsr0!, thanks for the Pull Request

The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.

  • The length of the commit message must be less than or equal to 72

To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page.

Read more about contributing to ESLint here

@akulsr0 akulsr0 changed the title fix: no-invalid-properties incorrect report when var is used inside calc fn fix: no-invalid-properties incorrect report when var used inside calc Aug 3, 2025
@eslint-github-bot eslint-github-bot bot added the bug Something isn't working label Aug 3, 2025
@nzakas
Copy link
Member

nzakas commented Aug 4, 2025

Thanks for taking a look at this. Because there was an existing issue already assigned to someone (docs), I'm closing this in favor of #227.

@nzakas nzakas closed this Aug 4, 2025
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

Rule Change: no-invalid-properties incorrectly reports for valid CSS with var() in functions
2 participants