microsoft / fast

The adaptive interface system for modern web experiences.
https://www.fast.design
Other
9.25k stars 590 forks source link

chore: remove legacy references to code climate #6767

Closed chrisdholt closed 1 year ago

chrisdholt commented 1 year ago

Pull Request

📖 Description

During the removal of fast-animation I found that the build was erroring out due to a code climate check/error. This PR removes legacy references to Code Climate. Currently we have references and checks we are partially running in the build, but Code Climate is not reporting on any valuable surface areas. This PR removes the remaining references to code climate as well as an old reference to Jest code coverage collection at the root which is no longer usable.

nicholasrice commented 1 year ago

During the removal of fast-element ...

fast-animation, right?

chrisdholt commented 1 year ago

During the removal of fast-element ...

fast-animation, right?

DOH. yes

radium-v commented 1 year ago

There's a script entry in the root-level package.json called coverage, as well as a few other jest-related scripts which don't use Lerna and are probably outdated.

chrisdholt commented 1 year ago

There's a script entry in the root-level package.json called coverage, as well as a few other jest-related scripts which don't use Lerna and are probably outdated.

Thanks - removed additional legacy stuff noted. This doesn't address everything but it does unblock and remove things which are known to not be leveraged currently.