-
Notifications
You must be signed in to change notification settings - Fork 6
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
Added styles rule #117
Added styles rule #117
Conversation
packages/eslint-plugin-fast-cli-migrate/src/rules/1.0.0-alpha.1--styles.md
Outdated
Show resolved
Hide resolved
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.
Something not quite right with the before/after I think. Looks like the html template and css styles got mixed together almost.
63a0798
to
80b48b9
Compare
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.
Approved pending the couple of things I suggested.
packages/eslint-plugin-fast-cli-migrate/src/rules/1.0.0-alpha.1--styles.md
Outdated
Show resolved
Hide resolved
packages/eslint-plugin-fast-cli-migrate/src/tests/files/1.0.0-alpha.1--styles.valid.ts
Outdated
Show resolved
Hide resolved
β¦1--styles.md Co-authored-by: Rob Eisenberg <EisenbergEffect@users.noreply.github.com>
β¦alpha.1--styles.valid.ts Co-authored-by: Rob Eisenberg <EisenbergEffect@users.noreply.github.com>
Pull Request
π Description
Adds a rule for
*.styles.ts
files.π« Issues
Continues work on #41
π©βπ» Reviewer Notes
This logic has been extracted into a utilities file because the exact same logic will apply to the next PR in this series which will deal with removing the function wrapper of the
*.template.ts
file.β Checklist
General