Closed surilindur closed 4 months ago
Totals | |
---|---|
Change from base Build 8927384227: | 0.0% |
Covered Lines: | 115 |
Relevant Lines: | 115 |
Just had a look, and everything looks good. Since the fragmenter has been released as well, feel free to make the new PR :-)
This is now finally ready for review! :snail:
Thanks!
Is a release for this needed already? (I guess major) Or are more changes planned?
Besides the work on multiple interfaces, I do not have any plans at the moment.
Looking at the commit history, I think the only breaking changes are around the (default) CSS version used for serving the content. Is it something that would require a major version bump? The other changes seem to be just a bugfix from Jitse, the various dependency updates and clean-up, but there are no changes to the SolidBench things themselves. Relocating index.ts
to lib/
should not break anything, since the package.json
path for it was also updated, and I doubt anyone directly imports it by path (I hope).
Here is a set of changes to:
@rubensworks/eslint-config
to version 3 and fixing the linter errors that popped uptsconfig.json
includes and excludesindex.ts
tolib/
Any feedback would be welcome! I will make a separate PR to use the new fragmenter version when it gets released.