-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split linting and loading #31
Conversation
78a3ec9
to
86ceba0
Compare
|
||
context('when `' + profileName + '` profile is loaded', () => { | ||
testProfile(profile); | ||
context('when `' + profileName + '` profile is loaded', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
String interpolation makes this easier
`when \`${profileName}\` profile is loaded`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
752d7d9
to
bca9746
Compare
1eca6bf
to
409ed02
Compare
409ed02
to
9dfca20
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed.
Closes #10
This lets us write tests directly against the linter instead of having to go via rule files. We can invent theoretical rules to ensure the mechanics are happy, aaand it's helped improve coverage.