Skip to content

Update README and remove legacy code - #2771

Merged
fvsch merged 5 commits into
mainfrom
fvsch/remove-legacy-code
Oct 10, 2023
Merged

Update README and remove legacy code#2771
fvsch merged 5 commits into
mainfrom
fvsch/remove-legacy-code

Conversation

@fvsch

@fvsch fvsch commented Oct 9, 2023

Copy link
Copy Markdown
Contributor

This PR is part of a small project to make the stackblitz/core repository more user-friendly and a better reflection of StackBlitz products and technology.

One part of this project is removing legacy code this is not used anymore and does not meaningfully represent the current feature set and capabilities of StackBlitz and its SDK and APIs.

This PR removes this legacy code. But that code is not deleted completely:

Finally, this PR rewrites this repo's README.md to remove links to posts from 2017 about the original StackBlitz technical architecture and Turbo. Instead, we link to our current docs, the StackBlitz blog, and the WebContainers website. I’ve also added links to a few select repositories.

Questions:

  • Should we link to the archive branch from the updated README in the main branch? I don’t know how much visibility we want to give it.
@fvsch
fvsch requested a review from sulco October 9, 2023 13:52

@sulco sulco left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we can leave the link to the archive out of readme for simplicity sake – and if someone's interested in digging into the history, they still can.

Looks great, thanks @fvsch!

@fvsch
fvsch merged commit 0125fd7 into main Oct 10, 2023
@fvsch
fvsch deleted the fvsch/remove-legacy-code branch October 10, 2023 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants