Skip to content

Clean project config - #822

Open
leonm1 wants to merge 8 commits into
mainfrom
clean-project-config
Open

Clean project config#822
leonm1 wants to merge 8 commits into
mainfrom
clean-project-config

Conversation

@leonm1

@leonm1 leonm1 commented Jun 9, 2021

Copy link
Copy Markdown
Member

Summary

Cleaning up some isolated exceptions in .eslintrc.js and unused deps

@leonm1
leonm1 force-pushed the clean-project-config branch from 0f8ae01 to 4e28703 Compare June 9, 2021 21:19
@leonm1
leonm1 marked this pull request as ready for review June 9, 2021 21:19
@leonm1
leonm1 enabled auto-merge (squash) June 9, 2021 21:20
Comment thread .eslintrc.js
Comment thread .eslintrc.js
Comment thread .eslintrc.js
Comment thread .eslintrc.js
@qlty-cloud-legacy

qlty-cloud-legacy Bot commented Jun 9, 2021

Copy link
Copy Markdown

Code Climate has analyzed commit 48869f0 and detected 8 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 4
Bug Risk 4

The test coverage on the diff in this pull request is 85.1% (0% is the threshold).

This pull request will bring the total coverage in the repository to 64.0% (0.6% change).

View more on Code Climate.

@leonm1
leonm1 disabled auto-merge June 9, 2021 21:28
@leonm1

leonm1 commented Jun 9, 2021

Copy link
Copy Markdown
Member Author

Not sure why Happo is deleting all our screenshots; will have to investigate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant