search

LivingPark-MRI / ppmi-scraper

An API to scrape PPMI data
MIT License
3 stars 10 forks source link

Merge python-app.yml and tests.yml? #33

Open yohanchatelain opened 2 years ago

yohanchatelain commented 2 years ago

We could merge those files and add cron schedule to python-app.yml to maintain one instead of two files. @mathdugre

mathdugre commented 2 years ago

Yes, I totally agree that we can merge those.

Eventually, we might want to separate the linting in a different action to only run on push and include more linting via pre-commit. Similar to livingpark-utils' linting. What do you think @yohanchatelain ?

yohanchatelain commented 2 years ago

Yes it is a good practice :+1: