Open krzkaczor opened 5 years ago
Proposed breaking changes:
do not implicitly initialize client but rather make it explicit like:
const client = new CodechecksClient();
instead of
import { client } from "@codechecks/client";
This will result in multiple benefits like a possibility to provide defaults for a check, client can execute checks in parallel etc.
rename directory dist to lib in final package
dist
lib
isFork should be a boolean in execution ctx
isFork
rename saveValue/getValue to saveJSON/getJSON
saveValue/getValue
saveJSON/getJSON
support scope setting. Useful for monorepos with multiple codechecks files. All codechecks artifacts and titles would share some scope
scope
by default, it exit with non-zero exit code if any report failed as discussed (https://github.com/codechecks/monorepo/issues/22)
I hope you'll remove __tests__ folder from the package as well :)
__tests__
Proposed breaking changes:
do not implicitly initialize client but rather make it explicit like:
instead of
This will result in multiple benefits like a possibility to provide defaults for a check, client can execute checks in parallel etc.
rename directory
dist
tolib
in final packageisFork
should be a boolean in execution ctxrename
saveValue/getValue
tosaveJSON/getJSON
support
scope
setting. Useful for monorepos with multiple codechecks files. All codechecks artifacts and titles would share somescope
by default, it exit with non-zero exit code if any report failed as discussed (https://github.com/codechecks/monorepo/issues/22)