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

docs: Removes recommendation for preact-cli #3816

Merged
merged 2 commits into from
Dec 21, 2022

Conversation

rschristian
Copy link
Member

At this point we can't really keep the recommendation for preact-cli in good faith. It's not in a good state, especially for users who are new to Preact. In lieu of another tool that we want to recommend in its place, the recommendation should be removed as (IMO) it's doing more harm than good.

Going to propose the same for the docs site.

@coveralls
Copy link

coveralls commented Dec 1, 2022

Coverage Status

Coverage remained the same at 99.531% when pulling b689f1f on docs/remove-cli-recommendation into 226d77d on master.

Copy link
Member

@marvinhagemeister marvinhagemeister left a comment

Choose a reason for hiding this comment

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

Good call! We should probably point folks to the vite preset, fresh or astro in the long run.

@developit developit merged commit 8a6d301 into master Dec 21, 2022
@developit developit deleted the docs/remove-cli-recommendation branch December 21, 2022 17:21
JoviDeCroock added a commit that referenced this pull request Jan 12, 2024
JoviDeCroock added a commit that referenced this pull request Jan 12, 2024
* backport #3871

* port test from #3884 functionality seems to work in v11

* backport #3880

* backport #3875

* backport #3862

* add todo for #3801

* backport #3868

* backport #3867

* backport #3863

* add todo for #3856

* backport #3844

* backport #3816

* backport #3888

* backport #3889
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.

None yet

5 participants