Run linting on the generated project in CI // add a working eslintconfig

embroider-build / app-blueprint

[pre-alpha™️] Blueprint for next generation of Ember apps

34 stars 11 forks source link

Run linting on the generated project in CI // add a working eslintconfig #84

Closed NullVoxPopuli closed 2 months ago

NullVoxPopuli commented 2 months ago

Unblocks CI verifying that https://github.com/embroider-build/app-blueprint/pull/83 is correct

Adds linting to the tests
Adds an eslint config
- ESLint 9
- Flat config (all ESLint 9 supports)
- GJS support

NullVoxPopuli commented 2 months ago

This has revealed that the upstream blueprint's eslintconfig is invalid.

I will be updating that in this PR

NullVoxPopuli commented 2 months ago

Yes! I agree -- to prove that point (because not everyone agreed), I'd like to get all the way to TS support, as in landing https://github.com/embroider-build/app-blueprint/pull/80, and then separating the blueprints -- which, I suspect will make for a much less weird index.js!