Skip to content

Conversation

@roninopf
Copy link
Contributor

🔹 Jira Ticket(s) if any

✏️ Description

  • split out IterableUtil
  • reorganizes some other files
  • no functional changes

@roninopf roninopf requested a review from Ayyanchira September 15, 2022 20:57
@codecov
Copy link

codecov bot commented Sep 15, 2022

Codecov Report

Merging #471 (77feaba) into master (e6c4349) will decrease coverage by 0.01%.
The diff coverage is 71.81%.

@@            Coverage Diff             @@
##           master     #471      +/-   ##
==========================================
- Coverage   62.48%   62.47%   -0.02%     
==========================================
  Files          66       67       +1     
  Lines        3916     3917       +1     
  Branches      454      453       -1     
==========================================
  Hits         2447     2447              
- Misses       1230     1233       +3     
+ Partials      239      237       -2     
Impacted Files Coverage Δ
...erable/iterableapi/IterableInAppMemoryStorage.java 56.25% <ø> (ø)
...ava/com/iterable/iterableapi/IterableUtilImpl.java 70.45% <70.45%> (ø)
...iterable/iterableapi/IterableInAppFileStorage.java 80.00% <76.19%> (ø)
...in/java/com/iterable/iterableapi/IterableUtil.java 70.00% <100.00%> (-0.10%) ⬇️
.../com/iterable/iterableapi/IterableRequestTask.java 79.47% <0.00%> (-2.19%) ⬇️
...ain/java/com/iterable/iterableapi/IterableApi.java 61.61% <0.00%> (+0.24%) ⬆️
...com/iterable/iterableapi/IterableInAppManager.java 87.14% <0.00%> (+0.47%) ⬆️
.../iterable/iterableapi/IterableActivityMonitor.java 95.08% <0.00%> (+3.27%) ⬆️

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

@Ayyanchira
Copy link
Member

Is there any slicing by certain functionality that the files are split up?

@roninopf roninopf merged commit ff412dc into master Sep 26, 2022
@roninopf roninopf deleted the jay/MOB-4852-reorganization branch September 26, 2022 23:35
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.

3 participants