Skip to content

[MOB-8617] Release/10.13.0 #230

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

Merged
merged 8 commits into from
Mar 31, 2022
Merged

[MOB-8617] Release/10.13.0 #230

merged 8 commits into from
Mar 31, 2022

Conversation

AliAbdelfattah
Copy link
Contributor

@AliAbdelfattah AliAbdelfattah commented Mar 31, 2022

Summary of Changes

  • Adds support for uploading debug symbols on Android to be used for crash deobfuscation
  • Adds Instabug Experiments APIs
  • Bumps Instabug native SDKs to v10.13
  • Fixes iOS platform calls not completing with void return type
  • Removes Generated Mock Files

Checklist

  • In addition to adding the new API to Dart, I have also added its implementation to both iOS and Android.
  • I have added at least 1 unit test in Dart for the new API and made sure the tests pass locally.
  • I have updated README.md to add the new API.

via-guy and others added 6 commits March 6, 2022 13:24
[MOB-8369] Fix `void` calls on iOS
* Add Experiments APIs (Flutter)

* Add Experiments APIs (Android)

* Add Experiments APIs (iOS)

* Update Changelog

* Add Experiments APIs tests
* Remove generated mock files

* Run `build_runner` before tests in CI
@codecov
Copy link

codecov bot commented Mar 31, 2022

Codecov Report

Merging #230 (c40a92c) into master (614f593) will increase coverage by 0.42%.
The diff coverage is 77.27%.

@@            Coverage Diff             @@
##           master     #230      +/-   ##
==========================================
+ Coverage   72.00%   72.42%   +0.42%     
==========================================
  Files          14       14              
  Lines         525      533       +8     
==========================================
+ Hits          378      386       +8     
  Misses        147      147              
Impacted Files Coverage Δ
lib/CrashReporting.dart 69.23% <33.33%> (ø)
lib/APM.dart 42.30% <66.66%> (ø)
lib/Instabug.dart 88.18% <100.00%> (+0.92%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 614f593...c40a92c. Read the comment docs.

shehabeldeenibrahim and others added 2 commits March 31, 2022 15:03
…220)

* Add gradle script to locate and zip obfuscation symbols files

* Add upload symbols task in build.gradle

* Add error handling to upload symbols script

* Add http request logic to upload symbols script

* Refactor upload symbols script

* Add clean up to upload symbols script

* Format upload symbols script

* Remove null checks in upload symbols script
Throw exceptions instead of repeated null checks

* Split get_config in upload symbols script

* Use function to combine paths in upload symbols script

* Add a check for instabugUploadEnable flag

* Add extra logs in upload symbols script

* Add code-style changes for upload symbols script

* Add support for gradle.properties in upload symbols script

* Add constants to upload symbols script

* Change variable definitions location priority
Look for upload variables in local props -> gradle props -> env variable

* Disable auto-upload by default

* Add support for env. and localprops for enableFlag
The script will look for the instabugUploadEnable in this order local->gradle->env

* Move upload flag check to the script

* Remove localProperties global variable
@AliAbdelfattah AliAbdelfattah self-assigned this Mar 31, 2022
@AliAbdelfattah AliAbdelfattah marked this pull request as ready for review March 31, 2022 13:11
@ymabdallah ymabdallah merged commit a5193c9 into master Mar 31, 2022
@ymabdallah ymabdallah deleted the release/10.13.0 branch March 31, 2022 16:10
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.

6 participants