Skip to content
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

feat: support class without a default constructor when using Jackson deserialization #358

Merged
merged 2 commits into from
Dec 26, 2023

Conversation

YongwuHe
Copy link
Collaborator

No description provided.

@YongwuHe YongwuHe force-pushed the feat/autoSerializer branch from 67740bf to 4e19217 Compare December 25, 2023 07:11
@YongwuHe YongwuHe changed the title feat: automatically select serializer feat: jackson no default creator Dec 25, 2023
Copy link

codecov bot commented Dec 25, 2023

Codecov Report

Attention: 850 lines in your changes are missing coverage. Please review.

Comparison is base (18fcfc6) 56.17% compared to head (594eb0a) 60.33%.
Report is 101 commits behind head on main.

Files Patch % Lines
...rex/inst/database/mongo/wrapper/ResultWrapper.java 4.54% 51 Missing and 12 partials ⚠️
...x/agent/bootstrap/model/ParameterizedTypeImpl.java 0.00% 52 Missing ⚠️
...database/mongo/wrapper/BulkWriteResultWrapper.java 0.00% 35 Missing ⚠️
.../database/mongo/WriteOperationInstrumentation.java 0.00% 31 Missing ⚠️
...nst/redisson/v3/wrapper/RedissonBucketWrapper.java 0.00% 31 Missing ⚠️
...java/io/arex/inst/netty/v3/common/NettyHelper.java 38.77% 26 Missing and 4 partials ⚠️
...x/inst/redisson/v3/wrapper/RedissonMapWrapper.java 0.00% 27 Missing ⚠️
...io/arex/inst/authentication/jwt/JJWTGenerator.java 50.00% 19 Missing and 6 partials ⚠️
...x/inst/redisson/v3/wrapper/RedissonSetWrapper.java 0.00% 25 Missing ⚠️
.../inst/redisson/v3/wrapper/RedissonListWrapper.java 0.00% 24 Missing ⚠️
... and 94 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #358      +/-   ##
============================================
+ Coverage     56.17%   60.33%   +4.16%     
- Complexity     1899     2441     +542     
============================================
  Files           263      319      +56     
  Lines         10097    11961    +1864     
  Branches       1148     1427     +279     
============================================
+ Hits           5672     7217    +1545     
- Misses         3913     4086     +173     
- Partials        512      658     +146     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@YongwuHe YongwuHe force-pushed the feat/autoSerializer branch from 4e19217 to df18c06 Compare December 25, 2023 07:48
@mr3
Copy link
Collaborator

mr3 commented Dec 25, 2023

suggest commit message: support class without a default constructor when using Jackson deserialization

@YongwuHe YongwuHe force-pushed the feat/autoSerializer branch from df18c06 to 9f6e7f5 Compare December 25, 2023 09:25
@YongwuHe YongwuHe force-pushed the feat/autoSerializer branch from 9f6e7f5 to 594eb0a Compare December 25, 2023 09:28
@YongwuHe YongwuHe changed the title feat: jackson no default creator feat: support class without a default constructor when using Jackson deserialization Dec 25, 2023
Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

2 New issues
0 Security Hotspots
88.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@YongwuHe YongwuHe merged commit 4c40e76 into main Dec 26, 2023
8 checks passed
@YongwuHe YongwuHe deleted the feat/autoSerializer branch December 26, 2023 02:04
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