Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Disallow inline styles via eslint #12803

Closed
2 tasks done
tay1orjones opened this issue Dec 6, 2022 · 0 comments · Fixed by #12924
Closed
2 tasks done

Disallow inline styles via eslint #12803

tay1orjones opened this issue Dec 6, 2022 · 0 comments · Fixed by #12924
Assignees
@tw15egan
Labels
role: dev 🤖
Milestone
2023 Q1

Comments

@tay1orjones
Copy link
Member

tay1orjones commented Dec 6, 2022 •
edited by tw15egan
Loading

Now that #12386 and #12580 are resolved, we should avoid using inline styles moving forward and enforce this via eslint rules.

Tasks
@tay1orjones tay1orjones moved this to ⏱ Backlog in Design System Dec 6, 2022
@tw15egan tw15egan mentioned this issue Jan 5, 2023
Merged
@tay1orjones tay1orjones added this to the 2023 Q1 milestone Jan 9, 2023
@tay1orjones tay1orjones moved this from ⏱ Backlog to 🚦 In Review in Design System Jan 11, 2023
@kodiakhq kodiakhq bot closed this as completed in #12924 Jan 18, 2023
@github-project-automation github-project-automation bot moved this from 🚦 In Review to ✅ Done in Design System Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tw15egan tw15egan

Labels
role: dev 🤖
Projects
Design System
Archived in project
Milestone
2023 Q1
Development

Successfully merging a pull request may close this issue.

test(eslint): update eslint to error if style prop is used tw15egan/carbon
2 participants
@tay1orjones @tw15egan