-
Notifications
You must be signed in to change notification settings - Fork 119
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
Allow runtime test discovery when sharding is disabled by not setting test-targets. This unblocks cucumber testing #822
Conversation
Codecov Report
@@ Coverage Diff @@
## master #822 +/- ##
============================================
+ Coverage 78.12% 78.30% +0.17%
Complexity 667 667
============================================
Files 149 149
Lines 3004 3014 +10
Branches 437 439 +2
============================================
+ Hits 2347 2360 +13
+ Misses 375 374 -1
+ Partials 282 280 -2 |
release_notes.md
Outdated
@@ -9,6 +9,7 @@ | |||
- [#805](https://github.com/Flank/flank/pull/805) Fix overlapping results. ([pawelpasterz](https://github.com/pawelpasterz)) | |||
- [#812](https://github.com/Flank/flank/issues/812) Convert bitrise macOS workflow to github action. ([piotradamczyk5](https://github.com/piotradamczyk5)) | |||
- [#799](https://github.com/Flank/flank/pull/799) Refactor Shared object by splitting it into smaller functions. ([piotradamczyk5](https://github.com/piotradamczyk5)) | |||
- [#822](https://github.com/Flank/flank/pull/822) Prevent set test targets is sharding is disabled and test-targets not set. ([adamfilipow92](https://github.com/adamfilipow92)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say it like this:
Allow runtime test discovery when sharding is disabled by not setting test-targets. This unblocks cucumber testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved, let's update the release note as mentioned.
30b0ebe
to
483a5f1
Compare
In CreateAndroidInstrumentationTest
Fixes #815
Checklist