Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

False negative check #17

Closed
asvetlov opened this issue Aug 2, 2019 · 3 comments
Closed

False negative check #17

asvetlov opened this issue Aug 2, 2019 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@asvetlov
Copy link

asvetlov commented Aug 2, 2019

PR aio-libs/aiohttp#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

  • We're using Polar.sh so you can upvote and help fund this issue.
  • We receive the funding once the issue is completed & confirmed by you.
  • Thank you in advance for helping prioritize & fund our backlog.
Fund with Polar
@webknjaz
Copy link
Member

webknjaz commented Aug 2, 2019

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

No files matching re.compile('CHANGES/(?P<issue_number>[^\./]+).(?P<fragment_type>bugfix|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
Copy link
Member

webknjaz commented Aug 2, 2019

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

@webknjaz
Copy link
Member

webknjaz commented Aug 2, 2019

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.

@webknjaz webknjaz added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Aug 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants