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

Add a deprecation notice #64

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add a deprecation notice #64

wants to merge 2 commits into from

Conversation

pepegc
Copy link

@pepegc pepegc commented Jan 7, 2023

Hi Rich, I stumbled upon this post on reddit. I've heard you say somewhere (couldn't find the video) that by now it makes no sense to continue working on this since there are other more advanced projects out there, most notable Threlte I believe.

You being you, I think it makes sense to add a deprecation notice so that new community members don't stumble upon this without context.

@vercel
Copy link

vercel bot commented Jan 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
svelte-cubed ❌ Failed (Inspect) Jan 7, 2023 at 2:58AM (UTC)

@ixxie
Copy link

ixxie commented Apr 11, 2023

Yes please merge this! I lost some time struggling with svelte-cubed before realizing Threlte exists...

@@ -1,3 +1,8 @@
# Warning: this project as been deprecated

Choose a reason for hiding this comment

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

There is a non zero-chance the Vercel build is failing because of the two top-level headers / H1s?

Maybe move it below the # svelte-cubed section, and also fix the little typo? 🙂

Suggested change
# Warning: this project as been deprecated
## Warning: this project has been deprecated

Great idea raising this, I've read that discussion thread and still didn't quite remember which is the more supported library now 😅

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

3 participants