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

Ratelimit Codecov for Labeller & Danger #5194

Merged
merged 1 commit into from Aug 7, 2022

Conversation

cortinico
Copy link
Member

Should solve the issue linked in:
#5193 (comment)

As those two workflows are on pull_request_target, we won't see in action in this PR. Hopefully once we rebase #5193 on top of the merge after this PR, this should solve the issue (it's a best guess fix though, I'm not sure it works 100% - API ref is here https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads).

@codecov
Copy link

codecov bot commented Aug 6, 2022

Codecov Report

Merging #5194 (d4b79d4) into main (6061f29) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #5194   +/-   ##
=========================================
  Coverage     84.96%   84.96%           
  Complexity     3638     3638           
=========================================
  Files           502      502           
  Lines         11985    11985           
  Branches       2260     2260           
=========================================
  Hits          10183    10183           
  Misses          689      689           
  Partials       1113     1113           

Help us with your feedback. Take ten seconds to tell us how you rate us.

@github-actions
Copy link

github-actions bot commented Aug 7, 2022

Warnings
⚠️ This PR is approved with no milestone set. If merged, it won't appear in the Detekt release notes.

Generated by 🚫 dangerJS against d4b79d4

@github-actions github-actions bot added the ci label Aug 7, 2022
@cortinico cortinico merged commit ee7949d into detekt:main Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants