Replies: 1 comment
-
Thank you for starting this discussion. We often forget the importance of a good development workflow in shaping a product.
This is something that we need the most as of now.
I will work on fixing current lint errors and after that this can be done too
This is something that can wait, I think due to reasons noted here #93 (review) We are focused too much on core-features and user feedbacks right now so I will leave it to the community for more input and discussions but so far your approach sounds good. |
Beta Was this translation helpful? Give feedback.
-
I have a PR draft #93 in progress and I ran into a few issues
Formatting and Linting.
The commands were not working so I made them target everything except node_modules and the build folders. There are ~86 errors in linting when I run pnpm lint and pnpm lint:fix won't fix them. We'll have to make sure the .eslintrc.js file is in good standing and then fix all the linting errors in the files. Those pesky errors can get annoying quickly but it will improve the workflow.
Enhancement: Adding a
pnpm check
command step to check for lint errors and formatting errors before the users make a PRContributions
I added the CONTRIBUTING.md file to guide and streamline contributions.
I added changeset to the pnpm workspace for version control but there are a few concerns noted in the draft from CI/CD and *.md commit code bloat that would need to be taken care of.
GitHub Actions for contributing
This is based on my current knowledge if anyone has more input please feel free to chat and/or give advice 💯
Beta Was this translation helpful? Give feedback.
All reactions