Duplication of tag definitions
It seems like this repo contain a duplicate set of definitions for what tags it considers special.
In github_activity.py there is a constant called TAGS_METADATA_BASE, but there is also tags.yaml. In conf.py it seems we rely on the python constant rather than the yaml file which I don't think is referenced anywhere.
I suggest we rely on the yaml file by default.
As a separate point, but if we rely on a YAML file to define what tags are valid, we could quite easily also allow for someone to pass a configuration to the github activity CLI etc allowing them to use custom tags and categories.
At this point, I'd like to have a CI category or similar. I'm not sure... Perhaps not listing any PRs marked CI would make more sense for a changelog? If something is CI, it doesn't matter at all to the changelog in my mind.
To Do
- Replace the python dictionary here: https://github.com/executablebooks/github-activity/blob/master/github_activity/github_activity.py#L17
- With a read-in from
tags.ymlhere: https://github.com/executablebooks/github-activity/blob/master/github_activity/tags.yaml
I think a YAML file sounds great to me - it would basically contain the mapping from "tags + keywords" -> categories, yeah?
Silly me, I just realized that tags.yml already exists. I updated the top comment with next steps