Please check if your PR fulfills the following requirements:
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (npm run build) was run locally and any changes were pushed
[x] Tests (npm test) were run locally and passed
[x] Prettier (npm run prettier) was run locally and passed
Pull request type
Please check the type of change your PR introduces:
- [ ] Bugfix
- [ ] Feature
- [ ] Refactoring (no functional changes, no api changes)
- [x] Build related changes
- [ ] Documentation content changes
- [ ] Other (please describe):
## What is the current behavior?
To build this project, there were two separate steps: compiling TypeScript and bundling through Rollup. I was running into some confusion as to why the `npm run watch` flag wouldn't work when having the tests running. Reason: the TS step would compile into ESM and not CJS which is later done by Rollup.
## What is the new behavior?
By adding the TypeScript Rollup plugin we can do two steps in one. This also removes the some of the typescript compiled files that were not exported by the package.
## Does this introduce a breaking change?
- [ ] Yes
- [ ] No
Pull request checklist
Please check if your PR fulfills the following requirements:
npm run build) was run locally and any changes were pushednpm test) were run locally and passednpm run prettier) was run locally and passedPull request type
Please check the type of change your PR introduces: - [ ] Bugfix - [ ] Feature - [ ] Refactoring (no functional changes, no api changes) - [x] Build related changes - [ ] Documentation content changes - [ ] Other (please describe): ## What is the current behavior? To build this project, there were two separate steps: compiling TypeScript and bundling through Rollup. I was running into some confusion as to why the `npm run watch` flag wouldn't work when having the tests running. Reason: the TS step would compile into ESM and not CJS which is later done by Rollup. ## What is the new behavior? By adding the TypeScript Rollup plugin we can do two steps in one. This also removes the some of the typescript compiled files that were not exported by the package. ## Does this introduce a breaking change? - [ ] Yes - [ ] NoTesting
n/a
Other information
n/a