intuit / postcss-themed

A PostCSS plugin for generating themes.
MIT License
62 stars 32 forks source link

New-line @theme causes build to hang #45

Closed kharrop closed 3 years ago

kharrop commented 3 years ago

Describe the bug

If a user has prettier installed with a max-line width, sometimes @theme [variable] can wrap to multiple lines. This causes the build to hang indefinitely with no indication of what's wrong. It's especially noticeable if the line in question has animation-related css.

/* prettier-ignore */ fixes this issue, but it can be pretty tedious to include this each time.

To Reproduce

Break up a line of css where @theme and the variable are on separate lines.

Expected behavior

Either it just works, or the build tells you where something is failing.

Screenshots

Desktop (please complete the following information):

Additional context

hipstersmoothie commented 3 years ago

:rocket: Issue was released in v2.4.2 :rocket: