Skip to content
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.

Convert to use ESLint for linting TypeScript, etc. (WIP) #174

Closed
wants to merge 3 commits into from

Conversation

@beausmith beausmith requested a review from benadida April 2, 2019 17:51
@CLAassistant
Copy link

CLAassistant commented Apr 2, 2019

CLA assistant check
All committers have signed the CLA.

@netlify
Copy link

netlify bot commented Apr 2, 2019

Deploy preview for vw-bmd ready!

Built with commit 0304433

https://deploy-preview-174--vw-bmd.netlify.com

Copy link
Contributor

@benadida benadida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@beausmith beausmith requested a review from benadida April 8, 2019 19:47
@beausmith
Copy link
Contributor Author

@benadida - I pushed a few more changes. Can you please re-review? 😄

Copy link
Contributor

@benadida benadida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@beausmith beausmith changed the title Convert to use ESLint for linting TypeScript, etc. (WIP) Convert to use ESLint for linting TypeScript, etc. Apr 10, 2019
@beausmith beausmith changed the title (WIP) Convert to use ESLint for linting TypeScript, etc. Convert to use ESLint for linting TypeScript, etc. (WIP) Apr 10, 2019
@beausmith
Copy link
Contributor Author

Something related to this config was causing Code Helper to take up 100% CPU. Thus this PR was on hold.

After a little more research, seems that it may have been related to @types/styled-components package:

Trying @types/styled-components@4.1.5 in another branch and it seems to be working.

beausmith added a commit that referenced this pull request Apr 16, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 17, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 17, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 17, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 17, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 17, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 17, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
beausmith added a commit that referenced this pull request Apr 23, 2019
In attempt to fix issues with high CPU usage of
VS Code due to bug in @types/styled-components, I
removed the node_modules folder and ran `yarn`
which updated this lock file.

More info:
#174 (comment)
@beausmith
Copy link
Contributor Author

Solved differently in: #329

@beausmith beausmith closed this May 9, 2019
@beausmith beausmith deleted the ts-eslint-parser branch May 9, 2019 09:06
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.

3 participants