-
Notifications
You must be signed in to change notification settings - Fork 463
Opinionated styling, package and code updates #28
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
Conversation
Thanks for your PR. |
Very useful change! |
@TheLartians
|
Sure, I'll create the new PR later today! |
Hey, thanks for this great template! It really helped me get started with Chrome extensions. Doing so, I've made a few adjustments to the starter that make it much more usable for me. I was wondering if some of these changes could be interesting for the original template as well.
I know that these are very opinionated changes, so feel free to simply ignore/close the PR if you are not interested. I could also split anything you find interesting into separate PRs, if you prefer.
Summary of changes
Date
classsum.ts
file into the example test, as it's not used for anything elsestrict
mode and allow compiling with errors (TS correctness will be checked in CI).vscode/tasks.json
as vscode will automatically detect tasks frompackage.json