System test 005 #2192

Merged
merged 2 commits into from Dec 14, 2016

Projects

None yet

5 participants

@m-allanson
Member
m-allanson commented Dec 12, 2016 edited

Which issue, if any, is this issue related to?

#2052

Is there anything in the PR that needs further explanation?

The config in this test uses every core rule (except no-empty-source) and the CSS will trigger a warning from each rule at least once. This generates a large amount of warnings, as a result the snapshot file ~600KB.

@m-allanson m-allanson Add system test 005
504316c
@ntwb
ntwb approved these changes Dec 13, 2016 View changes

LGTM

p.s. Includes a couple of doc tweaks that probably should be in a separate PR but meh, minor tweaks 😏

@jeddy3
jeddy3 approved these changes Dec 13, 2016 View changes

Oh wow, this must of taken an age to compile!

💯

@evilebottnawi

Fantastic

@davidtheclark
Contributor

Thanks @m-allanson! Seems like a perfect system test.

@davidtheclark davidtheclark merged commit 5405475 into master Dec 14, 2016

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@davidtheclark davidtheclark deleted the system-test-005 branch Dec 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment