Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Remove Storybook and related dependencies #16

Closed
1 of 4 tasks
brylie opened this issue Jan 25, 2022 · 4 comments · Fixed by #18
Closed
1 of 4 tasks

Remove Storybook and related dependencies #16

brylie opened this issue Jan 25, 2022 · 4 comments · Fixed by #18
Assignees
@sadn1ck
Labels
💻 aspect: code Concerns the software code in the repository ✨ goal: improvement Improvement to an existing feature help wanted Open to participation from the community 🟧 priority: high Stalls work on the project or its dependents 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work

Comments

@brylie
Copy link
Contributor

brylie commented Jan 25, 2022 •
edited by MuluhGodson
Loading

The Storybook dependency, while exciting, has led to complexity and dependency bloat in the Vocabulary Components. We are working to restore this project to good health, including updating all dependencies. We are considering removing Storybook in order to make dependency management easier. We can consider how to use Storybook via a separate repository, but want to avoid intermixing it into this project.

Task

  • open a discussion related to project development and documentation options that are similar to Storybook
  • remove Storybook content, files, and related dependencies
  • remove Storybook-related commands from package.json
  • remove Storybook-related documentation
@brylie brylie added 🟩 priority: low Low priority and doesn't need to be rushed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work ✨ goal: improvement Improvement to an existing feature 💻 aspect: code Concerns the software code in the repository labels Jan 25, 2022
@brylie brylie mentioned this issue Jan 25, 2022
Closed
4 tasks
@brylie
Copy link
Contributor Author

brylie commented Jan 25, 2022

@sadn1ck, would you like this task?

@sadn1ck
Copy link
Contributor

sadn1ck commented Jan 25, 2022

Sure @brylie!

@sadn1ck
Copy link
Contributor

sadn1ck commented Jan 25, 2022

Related discussion: #17

@brylie
Copy link
Contributor Author

brylie commented Jan 25, 2022

I believe we should prioritize this issue, since it is blocking #15

@brylie brylie added help wanted Open to participation from the community 🟧 priority: high Stalls work on the project or its dependents and removed 🟩 priority: low Low priority and doesn't need to be rushed labels Jan 25, 2022
@sadn1ck sadn1ck mentioned this issue Jan 25, 2022
Merged
7 tasks
brylie pushed a commit that referenced this issue Jan 25, 2022
Merge pull request #18 from sadn1ck/chore/deprecate-storybook
4b48f70 
fix(#16): remove storybook
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sadn1ck sadn1ck

Labels
💻 aspect: code Concerns the software code in the repository ✨ goal: improvement Improvement to an existing feature help wanted Open to participation from the community 🟧 priority: high Stalls work on the project or its dependents 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(#16): remove storybook
2 participants
@brylie @sadn1ck