Skip to content

refactor: Migrate to Vite - #11

Merged
ThisIsKirsch merged 2 commits into
github:mainfrom
rschristian:refactor/migrate-to-vite
Oct 23, 2023
Merged

ThisIsKirsch merged 2 commits into
github:mainfrom
rschristian:refactor/migrate-to-vite

Conversation

@rschristian

@rschristian rschristian commented Jul 29, 2023

Copy link
Copy Markdown
Contributor

We (the Preact team) no longer recommend using preact-cli; it's unfortunately not been able to get the development time it deserves and other tools offer a much better experience. We recommend Vite, and this updated template comes from our initializer create-preact.

I will say that I was quite disappointed to see someone took the time to copy my project template but (poorly) stripped out all of our reference material and slapped an Octocat in its place. That'd be fine if it were a custom-built template, but please don't strip out my work that's meant to help the end user and put your own branding in its place. It feels quite disrespectful.

@AbdullahWins AbdullahWins left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Job!

@ThisIsKirsch
ThisIsKirsch self-requested a review October 23, 2023 15:30
@ThisIsKirsch

ThisIsKirsch commented Oct 23, 2023

Copy link
Copy Markdown
Contributor

Hey @rschristian , we’re going to accept this PR, and we apologize for removing the content meant to help the end user. Thanks for making this PR!

@ThisIsKirsch
ThisIsKirsch merged commit 5b0004a into github:main Oct 23, 2023
@rschristian

Copy link
Copy Markdown
Contributor Author

Thanks!

@rschristian
rschristian deleted the refactor/migrate-to-vite branch October 23, 2023 20:41
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.

3 participants