Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

chore(ui): attempt to get ts linted #779

Closed
wants to merge 1 commit into from
Closed

Conversation

sarahscott
Copy link
Contributor

@sarahscott sarahscott commented Aug 15, 2020

Currently, eslint-plugin-svelte3 doesn't play nice with typescript. In order to use typescript in our components and also lint them, we need to preprocess before running through eslint.

see sveltejs/eslint-plugin-svelte3#62

@sarahscott sarahscott self-assigned this Aug 15, 2020
@xla
Copy link
Contributor

xla commented Aug 17, 2020

The committed package.json refers to a local location of eslint-plugin-svelte, therefore the tests fail and local checkout is also not helpful.

@sarahscott
Copy link
Contributor Author

@xla yup, it's still a draft. i'm working with a local version because the branch with a potential fix can't be simply pointed to with npm.

I'll ensure this passes tests & works locally before marking as ready for review.

@sarahscott sarahscott closed this Aug 19, 2020
@sarahscott sarahscott deleted the sos/get-ts-linted branch August 22, 2020 01:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants