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

removed catch predicate from integrations #14969

Merged
merged 1 commit into from
Aug 5, 2022

Conversation

Dave3of5
Copy link
Contributor

@Dave3of5 Dave3of5 commented Jun 5, 2022

Start of work removing the catch predicates for #14882.

@ErisDS ErisDS force-pushed the remove-catch-predicate-integrations branch from dc0deef to c588cb2 Compare August 5, 2022 11:53
@ErisDS ErisDS force-pushed the remove-catch-predicate-integrations branch from c588cb2 to 2bdbe52 Compare August 5, 2022 11:54
@codecov
Copy link

codecov bot commented Aug 5, 2022

Codecov Report

Merging #14969 (6ce3e8e) into main (6ce3e8e) will not change coverage.
The diff coverage is n/a.

❗ Current head 6ce3e8e differs from pull request most recent head 2bdbe52. Consider uploading reports for the commit 2bdbe52 to get more accurate results

@@           Coverage Diff           @@
##             main   #14969   +/-   ##
=======================================
  Coverage   52.73%   52.73%           
=======================================
  Files        1354     1354           
  Lines       86653    86653           
  Branches     9715     9715           
=======================================
  Hits        45693    45693           
  Misses      40909    40909           
  Partials       51       51           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ErisDS ErisDS changed the title #14882 Remove catch predicate from integrations removed catch predicate from integrations Aug 5, 2022
@ErisDS ErisDS merged commit 3c76172 into TryGhost:main Aug 5, 2022
@ErisDS
Copy link
Member

ErisDS commented Aug 5, 2022

Hey @Dave3of5 thank you for the PR and sorry for dropping the ball here, we got a bit of a backlog but are almost caught up again now. Have merged and this will go out in todays release. Thanks again 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants