/addon-wr

Eslint fixes #14

Closed
wants to merge 3 commits into
from

Conversation

Reviewers

@gregglind gregglind

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@pdehaan @gregglind
@pdehaan
Contributor

pdehaan commented Dec 7, 2017

No description provided.

addon/webextension/content-script.js
}
+ if (document.querySelector("#wanted")) {
@pdehaan

pdehaan Dec 7, 2017

Contributor

@gregglind Should this be .querySelector("#wanted") or .querySelector("#seen")?

@gregglind

gregglind Dec 7, 2017

Member

As it turns out, it's immaterial.

.eslintignore
addon/StudyUtils.jsm
addon/webextension/vendor/*
+dist
+test
@pdehaan

pdehaan Dec 7, 2017

Contributor

/test/ is ignored for now because there are a bunch of errors I didn't have time to dig through.

pdehaan added some commits Dec 6, 2017

@pdehaan
Fix a few ESLint issues
f9e6032
@pdehaan
A few more ESLint and package tweaks
333346b
@pdehaan
Actually lint all the files in addon/*
This commit was signed with a verified signature.
@pdehaan pdehaan Peter deHaan
GPG key ID: F0FC6C01C6305097 Learn about signing commits
e992e70

@pdehaan pdehaan closed this Dec 13, 2017

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