standard / standard-engine

:fire_engine: The guts of `standard` modularized for reuse
MIT License
145 stars 39 forks source link

Move APIs to `async`/`await` (+ fix tests failing due to ESM-based `standard`) #270

Closed voxpelli closed 3 years ago

voxpelli commented 3 years ago

What changes did you make? (Give an overview)

  1. I changed lintText and lintFiles to be async
  2. I adapted the tests to that + moved to import() than require() the cloned standard

Which issue (if any) does this pull request address?

Fixes #249

Is there anything you'd like reviewers to focus on?

Not really

voxpelli commented 3 years ago

I want to fix one thing before merging:

Changing cli() back to not be async, as else it will swallow errors thrown in it