Fix README.md contributors badge

[mehdichaouch]

Fix README.md contributors badge, causing the display of badge url code.

[colinmollenhour]

I don't know much about this badge but I think the "101" part of the url is basically meant to be dynamically replaced by the All Contributors bot so this PR would break that. Granted, it is already broken! 😄 #1614

[mehdichaouch]

I don't know much about this badge but I think the "101" part of the url is basically meant to be dynamically replaced by the All Contributors bot so this PR would break that. Granted, it is already broken! 😄 #1614

Oh I understand, by fixing this I maybe broke the bot in charge of changing this dynamically.

[mehdichaouch]

DON'T MERGE MY PULL REQUEST.

I read more about all-contributors-bot config, I think installation have to be checked and workflow to add contributor was not respected.

[Flyingmana]

@mehdichaouch yes there is somewhere a related ticket. We did not "install" the all contributors bot, because it needs write access to the repository, which could be used in a possible supply-chain-attack.

Means, it needs to be manually run and used to create PullRequests

[mehdichaouch]

@Flyingmana thank you for your answer.

So what about the badge, do you have a solution do fix it?
Should I close my PR or not?

[colinmollenhour]

Means, it needs to be manually run and used to create PullRequests

Is this something you could document somewhere very briefly like an internal wiki?

[fballiano]

what are we doing with this? it's really ugly to have a broken README

[Flyingmana]

Iam ok with merging it for now to fix the obvious error and take care of the automatism separately

[github-actions]

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
6 runs  ±0  4 ✔️ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit be65e79. ± Comparison against base commit 9b04ad2.