sanitizers / chronographer-github-app

Your severe chronographer robot who is watching you record all the news to change note files!
https://github.com/apps/chronographer
GNU General Public License v3.0
11 stars 3 forks source link

False negative check #17

Closed asvetlov closed 5 years ago

asvetlov commented 5 years ago

PR https://github.com/aio-libs/aiohttp/pull/3955 is marked red: ("Timeline protection: History fragments missing") but it has correct CHANGES/3551.misc.

@webknjaz please check what is going wrong?

Upvote & Fund

Fund with Polar

webknjaz commented 5 years ago

It's explained in the details section of https://github.com/aio-libs/aiohttp/runs/183618685:

No files matching re.compile('CHANGES/(?P[^./]+).(?Pbugfix|doc|feature|removal|trivial|vendor)(.\d+)?(.[^./]+)*$') pattern added

Since https://github.com/aio-libs/aiohttp/blob/master/pyproject.toml doesn't list the fragment types, the bot falls back to a hard-coded list defined here: https://github.com/sanitizers/chronographer-github-app/blob/0a112ca/chronographer/event_handlers.py#L258-L265

webknjaz commented 5 years ago

When the config has clearly defined types, the bot uses those for checking fragments. Here's a few examples:

webknjaz commented 5 years ago

Maybe I could use https://github.com/hawkowl/towncrier/blob/ecd438c/src/towncrier/_settings.py#L13-L21 to identify defaults in a more stable way.