Skip to content

Annotations on public and extension apis#18

Merged
kevinlind merged 1 commit intoadobe:dev-v2.0.0from
emdobrin:annotations
Jan 10, 2023
Merged

Annotations on public and extension apis#18
kevinlind merged 1 commit intoadobe:dev-v2.0.0from
emdobrin:annotations

Conversation

@emdobrin
Copy link
Copy Markdown
Contributor

@emdobrin emdobrin commented Jan 10, 2023

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@emdobrin emdobrin requested a review from kevinlind January 10, 2023 18:42
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 10, 2023 •
edited
Loading

Codecov Report

Merging #18 (73bb6f1) into dev-v2.0.0 (192f27a) will increase coverage by 5.63%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##             dev-v2.0.0      #18      +/-   ##
================================================
+ Coverage         76.03%   81.66%   +5.63%     
- Complexity          365      380      +15     
================================================
  Files                29       29              
  Lines              1577     1576       -1     
  Branches            225      225              
================================================
+ Hits               1199     1287      +88     
+ Misses              266      174      -92     
- Partials            112      115       +3
Flag Coverage Δ
functional-tests 66.18% <0.00%> (?)
unit-tests 76.08% <0.00%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...src/main/java/com/adobe/marketing/mobile/Edge.java 49.28% <0.00%> (+22.13%) ⬆️
...java/com/adobe/marketing/mobile/EdgeExtension.java 84.31% <ø> (+24.84%) ⬆️
...adobe/marketing/mobile/NetworkResponseHandler.java 81.00% <0.00%> (+2.26%) ⬆️
...va/com/adobe/marketing/mobile/ExperienceEvent.java 78.05% <0.00%> (+2.44%) ⬆️
.../marketing/mobile/StoreResponsePayloadManager.java 77.27% <0.00%> (+3.03%) ⬆️
...ain/java/com/adobe/marketing/mobile/EdgeState.java 98.41% <0.00%> (+3.17%) ⬆️
...a/com/adobe/marketing/mobile/EdgeHitProcessor.java 81.82% <0.00%> (+4.55%) ⬆️
...in/java/com/adobe/marketing/mobile/EventUtils.java 87.50% <0.00%> (+6.25%) ⬆️
...java/com/adobe/marketing/mobile/ConsentStatus.java 91.30% <0.00%> (+8.70%) ⬆️
...com/adobe/marketing/mobile/EdgeNetworkService.java 73.40% <0.00%> (+9.04%) ⬆️
... and 2 more

@kevinlind kevinlind merged commit 1fd8d54 into adobe:dev-v2.0.0 Jan 10, 2023
@emdobrin emdobrin deleted the annotations branch January 10, 2023 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevinlind kevinlind kevinlind approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@emdobrin @kevinlind