hsf-training / hsf-training-cpp-webpage

C++ training
https://hsf-training.github.io/hsf-training-cpp-webpage
6 stars 11 forks source link

Pre-commit checks are failing #22

Open amanmdesai opened 1 year ago

amanmdesai commented 1 year ago

see for instance #21

klieret commented 1 year ago

Right. This is because I've enabled the bot but the config is missing. I thought I had automatically added that to every repository, but somehow that apparently didn't work. Strange.

klieret commented 1 year ago

@amanmdesai Btw, it's not clear if this training module and the "intermediate" one are still in active development. There is consistent effort on https://github.com/hsf-training/cpluspluscourse however

amanmdesai commented 1 year ago

Right. This is because I've enabled the bot but the config is missing. I thought I had automatically added that to every repository, but somehow that apparently didn't work. Strange.

I see. Thanks.

amanmdesai commented 1 year ago

@amanmdesai Btw, it's not clear if this training module and the "intermediate" one are still in active development. There is consistent effort on https://github.com/hsf-training/cpluspluscourse however

Oh okay. Thanks for the link.

amanmdesai commented 1 year ago

Right. This is because I've enabled the bot but the config is missing. I thought I had automatically added that to every repository, but somehow that apparently didn't work. Strange.

Hi @klieret, I was just curious to learn how the bot works and how its set-up. Could you please point me to a reference/code on how this can be done on a repo?

Thanks.

klieret commented 1 year ago

Hi @amanmdesai : https://pre-commit.com/ is a good resource for this. This tells you how to set it up in your repository and your local git. https://pre-commit.ci/ is then the version that then automatically also runs these tests as a github CI system

amanmdesai commented 1 year ago

Thanks very much for the references.

stale[bot] commented 1 year ago

This issue or pull request has been automatically marked as stale because it has not had recent activity. Please manually close it, if it is no longer relevant, or ask for help or support to help getting it unstuck. Let me bring this to the attention of @klieret @wdconinc @michmx for now.