Skip to content

Add ability for the caller to handle the firebase failures#430

Merged
AliSoftware merged 3 commits intotrunkfrom
firebase-test-return-url
Nov 30, 2022
Merged

Add ability for the caller to handle the firebase failures#430
AliSoftware merged 3 commits intotrunkfrom
firebase-test-return-url

Conversation

@AliSoftware
Copy link
Contributor

@AliSoftware AliSoftware commented Nov 30, 2022

Especially so they can use buildkite-annotate and/or do other steps to report the failure before bailing

To Test

See this PR in WordPress-Android where this new feature has been tested and used, in particular:

  • This Buildkite run where an intentional failure was triggered and you can see the annotation
  • This Buildkite run where the intentional failure was reverted and you can see the Instrumented Tests step green and no annotation.

Especially so they can use `buildkite-annotate` and/or do other steps to report the failure before bailing
@AliSoftware AliSoftware force-pushed the firebase-test-return-url branch from de1a465 to d1eff86 Compare November 30, 2022 12:44
@AliSoftware AliSoftware self-assigned this Nov 30, 2022
@AliSoftware AliSoftware added the enhancement New feature or request label Nov 30, 2022
@AliSoftware AliSoftware requested a review from a team November 30, 2022 13:20
@AliSoftware AliSoftware marked this pull request as ready for review November 30, 2022 13:20
@ParaskP7 ParaskP7 self-assigned this Nov 30, 2022
Copy link

@ParaskP7 ParaskP7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome @AliSoftware , I really longed for this!

I reviewed and tested both, this and its associated WPAndroid PR, everything LGTM! 🥇 ❤️ 🚀

Copy link
Contributor Author

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small typo in CHANGELOG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants