search

graphile / crystal-pre-merge

Repository moved
https://github.com/graphile/crystal
39 stars 8 forks source link

Speed up GitHub Actions #492

Closed CSNWEB closed 1 year ago

CSNWEB commented 1 year ago

@benjie I think I am done with my rework, please have a look again, whenever you have time. Thanks!

benjie commented 1 year ago

I've merged the various tests into three main files: lint.yml for linting (ESLint, prettier, depcheck), test.yml for the tests, and posttest.yml for all the post tests. They all still run separately and in parallel, but this felt easier to manage to me.

benjie commented 1 year ago

Also apparently I never ran prettier on yml files before - that was a mistake! Now I do, which means we also get notified if their syntax is invalid!

CSNWEB commented 1 year ago

Ah yes merging them into single files makes a lot of sense 👍 .