Remove history, core contributor status, and people.yaml

[nedbat]

This removes a lot...

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.39% ⚠️

Comparison is base (a6490a3) 89.71% compared to head (7c2499e) 89.33%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #251      +/-   ##
==========================================
- Coverage   89.71%   89.33%   -0.39%     
==========================================
  Files          73       73              
  Lines        4327     4152     -175     
  Branches      452      429      -23     
==========================================
- Hits         3882     3709     -173     
+ Misses        410      409       -1     
+ Partials       35       34       -1     

Files Changed	Coverage Δ
openedx_webhooks/bot_comments.py	100.00% <ø> (ø)
openedx_webhooks/labels.py	100.00% <ø> (ø)
openedx_webhooks/lib/webhooks_data/models.py	96.49% <ø> (ø)
openedx_webhooks/tasks/pr_tracking.py	97.32% <ø> (-0.06%)	⬇️
tests/test_pull_request_closed.py	100.00% <ø> (ø)
openedx_webhooks/info.py	98.60% <100.00%> (+0.75%)	⬆️
tests/test_data.py	100.00% <100.00%> (ø)
tests/test_info.py	100.00% <100.00%> (ø)
tests/test_pull_request_opened.py	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[feanil]

I was a little hesitant to remove the core_committer PR comment since I think there is a version of that which could be useful but given that we'd have to do some work in SF to start tracking that info again, I don't think it's useful to keep for now. We don't have a plan to track that data there any time soon since it adds a bunch of on-going overhead.

[nedbat]

One option we could explore: keep people.yaml specifically for the core contributor status. It works now, and doesn't need to be updated that often.

[nedbat]

OK, as discussed in Slack, we aren't going to keep people.yaml, which has a more complicated schema than we need anyway. I don't know when we might get CC data from Salesforce, but when we do, we can put back some parts of this code. It feels a little odd to remove it knowing that we might want it back in the future, but we know where it is and we can come back to get it.