intuit / postcss-themed

A PostCSS plugin for generating themes.

MIT License

62 stars 32 forks source link

Added new test cases to increase code coverage #31

Closed ratnamal closed 3 years ago

ratnamal commented 3 years ago

What Changed

Add new tests in legacy and modern test files Fixed an issue where field forceSingleTheme is taking boolean value instead of a string in a few tests in legacy.test.ts file

Why

To increase code coverage

Issue: #18

Todo:

[X] Add tests
[X] Add docs
[X] Add yourself to contributors (run yarn contributors:add)

ratnamal commented 3 years ago

Original coverage: Coverage on master branch

Coverage after adding tests:

ratnamal commented 3 years ago

Thanks for the contribution! This is looking great.

Can you fix the conflicts (we had some new contributors), and then I can merge.

@tylerkrupicka Fixed the merge conflicts. Thank you for the review

hipstersmoothie commented 3 years ago

:rocket: PR was released in v2.1.0 :rocket: