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

Implement specs #155

Closed
Arcanemagus opened this issue Jan 14, 2016 · 8 comments
Closed

Implement specs #155

Arcanemagus opened this issue Jan 14, 2016 · 8 comments

Comments

@Arcanemagus
Copy link
Member

It looks like this linter does not currently have any specs written for it, adding it to the list being maintained here for tracking.

Specs do not need to be super detailed, as they should mainly test whether the package is properly running the linter plugin and parsing its messages. Leave testing of the linter functionality to the author of the linter.

If you need any help feel free to ask!

@casesandberg
Copy link
Contributor

I started work on that last night on the tests branch https://github.com/AtomLinter/linter-jscs/tree/tests. I have the very basic specs that all the other linters have, but I am going to put some more robust ones in there tonight before merging it in.

@Arcanemagus
Copy link
Member Author

I just blanket submitted this to all repos that didn't have a spec folder 😉

You should check out that tracking issue lol... the list is huge 😞.

@casesandberg
Copy link
Contributor

Yeah I figured 👍

If I am ever in a testing mood one day I might go through and just dump basic ones in there for a bunch 🎱

@Arcanemagus
Copy link
Member Author

Yea, I'll probably be getting to them eventually, but I think I'm going to focus on AtomLinter/Meta#9 first, since completely broken is worse than not being tested.

@casesandberg
Copy link
Contributor

I might be able to take over https://github.com/AtomLinter/linter-jsxhint too, I am interested in anything JavaScript / react related

@casesandberg
Copy link
Contributor

Will look into it tonight!

@Arcanemagus
Copy link
Member Author

Actually it looks like JSXHint itself is deprecated, so that package might end up getting removed: https://github.com/STRML/JSXHint

@casesandberg
Copy link
Contributor

Ah, well there you go

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants