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

Storybook husky #729

Closed
wants to merge 6 commits into from
Closed

Storybook husky #729

wants to merge 6 commits into from

Conversation

kamui-fin
Copy link
Contributor

This PR introduces two new dev-experience enhancing tools: Storybook and Pre-commit hooks.

Pre-Commit Hooks

Simply formats and lints only the changed (staged) files to avoid pointless CI runs and saves time overall. Runs automatically on commit.

Storybook

Allows developers to develop components seamlessly in an isolated environment, bettering the quality of the components overall. My suggestion is to make writing storybook stories mandatory for new components for robustness. Would love to hear feedback on that @cubetastic33

@kamui-fin kamui-fin requested a review from a team as a code owner October 31, 2023 04:22
@kamui-fin kamui-fin enabled auto-merge October 31, 2023 04:22
@kamui-fin
Copy link
Contributor Author

Okay this was not intended but my commits for temporary work on the TRPC optimizations got mixed in. Will resolve this.

@github-actions
Copy link

@github-actions
Copy link

@cubetastic33 cubetastic33 marked this pull request as draft October 31, 2023 13:13
auto-merge was automatically disabled October 31, 2023 13:13

Pull request was converted to draft

@cubetastic33
Copy link
Member

Making this draft until the TRPC commits are reverted on this branch so we don't accidentally merge

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.

2 participants