Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build with SvelteKit packager #315

Merged
merged 2 commits into from
Jan 5, 2024
Merged

Build with SvelteKit packager #315

merged 2 commits into from
Jan 5, 2024

Conversation

theodorejb
Copy link
Contributor

  • Components are now written in TypeScript, avoiding the need to maintain and test separate type definitions.
  • Migrated docs site to use SvelteKit and Bootstrap 5 (preview).
  • Switched from Prism to Highlight.js for code syntax highlighting.
  • Enabled Prettier to autoformat code.
  • Switched from Jest to Vitest for tests.
  • Separate gh-pages branch is now used to deploy docs so autogenerated files don't have to be committed to master branch (this may require updating the GitHub Pages Source repository setting).
  • Added contributing instructions file.

These changes should make it a lot easier to build and test the project, lowering the barrier for contributions and future maintenance.

Closes #313, closes #311

* Components are now written in TypeScript, avoiding the need to maintain and test separate type definitions.
* Migrated docs site to use SvelteKit and Bootstrap 5.
* Switched from Prism to Highlight.js for code syntax highlighting.
* Enabled Prettier to autoformat code.
* Switched from Jest to Vitest for tests.
* Separate `gh-pages` branch is now used to deploy docs so autogenerated files don't have to be committed to master branch.
* Added contributing instructions file.

These changes should make it a lot easier to build and test the project, lowering the barrier for contributions and future maintenance.

Closes #313, closes #311
@mcmxcdev
Copy link

Amazing work @theodorejb !

This PR would clean up lots of issues with this repo, @Cweili do you think you could find the time to review and get this merged? Would be highly appreciated :)

@Cweili
Copy link
Owner

Cweili commented Jan 5, 2024

@theodorejb Excellent work! I believe what we currently lack is some test cases to achieve complete coverage.

@Cweili Cweili mentioned this pull request Jan 5, 2024
@theodorejb
Copy link
Contributor Author

@Cweili Thanks! I don't believe coverage has actually decreased; it's that .svelte files were previously excluded from the report, and now they are not. So the coverage is simply more accurate now.

@Cweili Cweili merged commit 9b1607e into Cweili:master Jan 5, 2024
2 of 3 checks passed
@Cweili
Copy link
Owner

Cweili commented Jan 5, 2024

Released with v4. Thanks!

@theodorejb theodorejb deleted the svelte-kit branch January 5, 2024 15:46
@marekdedic marekdedic mentioned this pull request Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update package.json for vite5 / svelte 3 Add way to locally preview changes
4 participants