[added] lint to CI
ClosedPublic

Authored by appsforartists on Mar 22 2017, 10:33 PM.

Details

Summary

If our conversion to bazel works, this will probably become obsolete. But, I noticed it was only blocked on https://github.com/palantir/tslint/issues/2388, so I thought I'd give it another go.

Closes https://github.com/material-motion/material-motion-js/issues/161
Closes https://github.com/palantir/tslint/issues/2388

Diff Detail

Repository
R13 material-motion/material-motion-js
Lint
Lint OK
Unit
No Unit Test Coverage
Restricted Application added a project: Material Motion. · View Herald Transcript
appsforartists planned changes to this revision.Mar 22 2017, 10:33 PM
Restricted Application failed to build Restricted Buildable!Mar 22 2017, 10:35 PM

Using current versions of dependencies

appsforartists retitled this revision from [wip] lint failures to CI to [added] lint to CI.Sep 28 2017, 2:00 AM
appsforartists edited the summary of this revision. (Show Details)
Restricted Application completed building Restricted Buildable.Sep 28 2017, 2:01 AM
appsforartists edited the summary of this revision. (Show Details)

Ensuring lint-ci runs during CI

Restricted Application failed to build Restricted Buildable!Sep 28 2017, 2:08 AM

Rebased with fixed tests

Restricted Application completed building Restricted Buildable.Sep 28 2017, 2:37 AM
Restricted Application completed building Restricted Buildable.Sep 28 2017, 2:42 AM
featherless accepted this revision.Oct 6 2017, 9:13 AM
This revision is now accepted and ready to land.Oct 6 2017, 9:13 AM
This revision was automatically updated to reflect the committed changes.