Publish SKY UX TSLint rules as separate package
Closed this issue · 2 comments
Blackbaud-IsaacAggrey commented
Expected behavior
npm install @skyux/tslint-rules
- doesn't exist ;)
Actual behavior
Currently, TSLint rules are only provided by skyux-builder. Plain NPM libraries currently might not use skyux-builder. Different discussion is if one day they will?
Implementation
It would model something like AirBnb's TSLint:
- NPM package - https://www.npmjs.com/package/tslint-config-airbnb
- repo - https://github.com/progre/tslint-config-airbnb
Impact
skyux-builder
would instead pull in its TSLint rules as a dependency by skyux-builder
.
Consumers would need to change their tslint.json
from @blackbaud/skyux-builder/tslint
to @skyux/tslint-rules/tslint
Blackbaud-SteveBrush commented
We would more than likely move this to the @skyux-sdk
namespace, to keep it consistent with other development dependencies.
Blackbaud-AlexKingman commented
Closing due to inactivity. If there is still a need for this functionality, we'd be happy to take a contribution.