Add black as dependency at install

[albertvillanova]

Add black as dependency at install.

Note that, besides make quality, black is also used in style_doc.py:

doc-builder/src/doc_builder/style_doc.py

Line 21 in e001746

import black

[mishig25]

thanks a lot!

[akx]

This has apparently not been released yet?

[albertvillanova]

It is true that no more official releases have been done since the last v.0.2.0 on Mar 25, 2022: https://github.com/huggingface/doc-builder/releases/tag/v0.2.0

On the other hand, we use the latest code by checking out this repository from the main branch.

Maybe some of the maintainers can give more insights...

[akx]

On the other hand, we use the latest code by checking out this repository from the main branch.

That's unfortunately not the case for Huggingface's own repositories, where I had to add an explicit hidden dependency for black because of this:

huggingface/accelerate#2400
huggingface/peft#1421

[albertvillanova]

I meant in our CIs: https://github.com/huggingface/datasets/blob/f9975f636542df7f95c27065ea93147440d690b7/.github/workflows/build_documentation.yml#L13

[akx]

That may be the case for datasets, but evidently not accelerate and peft.

The @main in the uses reference feels ill-advised to me, though – any change (accidental, malicious, etc.) merged into main here will immediately cause problems in every other PR everywhere else. A tried and tested @release branch maybe, if you don't want to lock into versions, but otherwise the GitHub Actions documentation lays down some good guidelines.