Skip to content

fix(eslint-plugin): Add missing valid-styles property validators#1708

Open
skovhus wants to merge 1 commit into
facebook:mainfrom
skovhus:fix/eslint-plugin-property-allowlist
Open

fix(eslint-plugin): Add missing valid-styles property validators#1708
skovhus wants to merge 1 commit into
facebook:mainfrom
skovhus:fix/eslint-plugin-property-allowlist

Conversation

@skovhus

@skovhus skovhus commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

What changed / motivation ?

Adds valid-styles validators for CSS properties already present in StyleX types and clean-order priorities, including logical scroll spacing, timelines, SVG paint/stroke properties, and logical overflow/overscroll properties.

Linked PR/Issues

n/a

Additional Context

Adds tests proving representative valid usage is accepted and constrained enum values still reject invalid inputs.

Screenshots, Tests, Anything Else

Pre-flight checklist

@skovhus skovhus requested review from mellyeliu and nmn as code owners June 11, 2026 18:39
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 11, 2026
@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
stylex Skipped Skipped Jun 11, 2026 6:39pm

Request Review

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant