search

google / postcss-rename

Replace class names based on a customizable renaming scheme.
Apache License 2.0
131 stars 18 forks source link

Add Bazel test target for JS tests #12

Closed rzhw closed 4 years ago

rzhw commented 4 years ago

Closes Affects #9

nex3 commented 4 years ago

WDYT about moving this away from Bazel and towards infrastructure that's more standard for the open-source world, like invoking tests directly from Travis?

rzhw commented 4 years ago

I think while the Bazel infrastructure exists it's helpful to have more coverage in the form of Bazel test target(s). It doesn't preclude us from adding the JS tests directly to CI though—I'll change this PR to not mark #9 as fixed