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

Remove styleguide #329

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from
Draft

Remove styleguide #329

wants to merge 3 commits into from

Conversation

TyHil
Copy link
Member

@TyHil TyHil commented Nov 22, 2024

Overview

I don't think anyone knows how to use the react-styleguidist stuff. I was added in a couple commits in 2022 and hasn't been touched since then. The main benefit I see of it is it allows README documents to render components they are documenting which would be nice.

What Changed

Removed the styleguide and styleguidistDocs folders, the styleguide.config.js file, and a bunch of packages in package.json. I'm not sure how many of the packages being removed were needed for react-styleguidist and how many were just unused.

Copy link

vercel bot commented Nov 22, 2024

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

Name Status Preview Comments Updated (UTC)
utd-trends ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 6:50pm

@TyHil
Copy link
Member Author

TyHil commented Nov 22, 2024

@NishilJ if you have any luck getting react-styleguidist to work we can discard this PR.

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.

1 participant