-
Notifications
You must be signed in to change notification settings - Fork 0
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
(Feature) Setup #1
Comments
Branch issue-1-Setup created! |
Why? I would rather change that to setup github action for linting. I'm not a fan of tasks running when I do my commits |
Is there a reason for this? IDK but I have kinda just been sticking with NPM for most of my tasks lately and have not encounterd any issues |
Also should this maybe be split in to a few issues. Some of these can be done in pararell. |
I agree that there should be GitHub action checking this, but I like to catch errors and warnings as soon as possible, so that's why I like having formatting and linting checks on pre-commit Git hooks. Not sure what's the easiest way to have this conveniently be turned on and off according to preferences. 🤔
Not sure why I put this here,
I kinda want to do this myself in one go as I'm going to document this process – might put post migration process online somewhere too – and use it at work if possible. 🙂 Most of the items on the checklist are minor issues anyway; connecting them all is a bit more tricky. |
Detailed Description
General
@
aliasFormatting and linting
PWA
Testing
Clean up
eslint-disable
s, if possibleindex.ts
fileDisregarded
Change fromNo advantage of usingnpm
toyarn
yarn
, at least for our use case.Add stylelintNo need (at least initially) since we want to use Tailwind CSS.Seldom you want theindex.html
title referenceindex.html
to be the same as thename
inpackage.json
.Context
It's easier to develop once all this stuff is in place.
Possible Implementation
Complete all tasks above.
The text was updated successfully, but these errors were encountered: