Repositories with non-conforming MAINTAINERS file

[smola]

Some of our repositories contain a maintainers file that does not conform to what our docs specify.

That is, one of the following:

• The name of the file is not exactly MAINTAINERS.
• The format is not exactly Full Name <email> (@github_handle) per line.

Please, note that modifications to this, as fancy as they might look, just add more corner cases to be handled for any kind of scripting that relies on this. Let's use a single format.

These are the affected repositories that are using MAINTAINERS.md:

• src-d/apollo
• src-d/awesome-machine-learning-on-source-code
• src-d/datasets
• src-d/hercules
• src-d/jgscm
• src-d/kmcuda
• src-d/lookout-sdk-ml
• src-d/minhashcuda
• src-d/ml
• src-d/style-analyzer
• src-d/tmsc
• src-d/vecino
• src-d/wmd-relax

These are using (github_handle) instead of (@github_handle):

• src-d/blog
• src-d/ci
• src-d/docs
• src-d/landing
• src-d/talks (archived repository)

[smola]

cc @mcuadros @vmarkovtsev @dpordomingo

[vmarkovtsev]

Ack, we will fix

[zurk]

@vmarkovtsev priority?

[smola]

Thanks @vmarkovtsev!

[vmarkovtsev]

@zurk Background

[dpordomingo]

@smola I felt the necessity of converting the repo list into a todo list, to let the people update.

• I updated blog, landing and docs
• I could not update talks because it is read-only 🤷‍♂️

[zurk]

Also:

• https://github.com/src-d/ml-core
• https://github.com/src-d/ml-mining
• https://github.com/src-d/eee-commit-time-series

Going to fix it today.

[bzz]

@smola shall we do something similar here or in a separate issue for the repositories without MAINTAINERS file? As there are some, just from the top of the head:

• https://github.com/src-d/go-oniguruma
• https://github.com/src-d/go-license-detector
• https://github.com/src-d/models
• https://github.com/src-d/terraform-provider-online
• https://github.com/src-d/terraform-provisioner-online-rescue
• https://github.com/src-d/conferences
• https://github.com/src-d/superset-compose

[smola]

@bzz A separate issue is better. The initial purpose here was reducing special cases to be handled by tooling. No MAINTAINERS means there is no maintainer assigned, which is also a problem, but a separate one.

[zurk]

@vmarkovtsev can you take a look for all these open PRs above? Under This was referenced 11 days ago title

[zurk]

@smola I think we can close this one.

[smola]

• Fixed on src-d/ci src-d/ci#100
• Thanks @zurk!

[smola]

• Fixed on src-d/ci src-d/ci#100
• Thanks @zurk!