social.benbrown.com

Jan 18th 23 8:27 pm

@benbrown i cleaned up your codebase github.com/benbrown/shuttlecra

benbrown
benbrown@social.benbrown.com
selfagency
selfagency@kibitz.cloud

@benbrown if you're interested, i'd be happy to convert it to typescript if and when these changes get merged

selfagency
selfagency@kibitz.cloud

@benbrown btw, is there a way to run locally for dev purposes w/o SSL?

benbrown
benbrown@social.benbrown.com

@selfagency@kibitz.cloud i run it behind ngrok.

benbrown
benbrown@social.benbrown.com

@selfagency@kibitz.cloud no thanks i don't want to introduce build steps

selfagency
selfagency@kibitz.cloud

@benbrown totally reasonable

selfagency
selfagency@kibitz.cloud

@benbrown do you want any github actions to validate code, scan for security vulnerabilities, etc.?

benbrown
benbrown@social.benbrown.com

@selfagency@kibitz.cloud would love to enforce the linter rules on PR. will turn on dependabot eventually to monitor for problems in the tree.

selfagency
selfagency@kibitz.cloud

@benbrown ok, the action is in the pr. you just need to set the branch rules for `main` in the github settings to require that the `eslint` and `prettier` checks pass before merge.

selfagency
selfagency@kibitz.cloud

@benbrown i'll figure out handlebars linting in another PR, i already overloaded this one

benbrown
benbrown@social.benbrown.com

@selfagency@kibitz.cloud yeah let me crunch on this one for a bit. I don't have a ton of time to work on shuttlecraft during the week.

selfagency
selfagency@kibitz.cloud

@benbrown if you'd prefer I can go back and break it into smaller PRs

benbrown
benbrown@social.benbrown.com

@selfagency@kibitz.cloud no I think it will be fine