Closed mrehan27 closed 10 months ago
Pull request title looks good 👍!
If this pull request gets merged, it will cause a new release of the software. Example: If this project's latest release version is 1.0.0
. If this pull request gets merged in, the next release of this project will be 1.0.1
. This pull request is not a breaking change.
All merged pull requests will eventually get deployed. But some types of pull requests will trigger a deployment (such as features and bug fixes) while some pull requests will wait to get deployed until a later time.
Merging #246 (19fce99) into main (0587efa) will increase coverage by
1.04%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #246 +/- ##
============================================
+ Coverage 49.76% 50.80% +1.04%
- Complexity 237 249 +12
============================================
Files 108 108
Lines 2781 2781
Branches 364 364
============================================
+ Hits 1384 1413 +29
+ Misses 1282 1250 -32
- Partials 115 118 +3
Files Changed | Coverage Δ | |
---|---|---|
...stomer/messaginginapp/gist/presentation/GistSdk.kt | 13.38% <100.00%> (+13.38%) |
:arrow_up: |
Build available to test
Version: fix-in-app-listeners-concurrent-modification-crash-SNAPSHOT
Repository: https://s01.oss.sonatype.org/content/repositories/snapshots/
I tried different approaches and Shahroz's suggestion for trying thread safe collections looked the best option. I have used CopyOnWriteArrayList which works well and added tests. The tests passes with current implementation and fail if we run them on previous implementation.
fixes: https://github.com/customerio/customerio-android/issues/245
Changes