Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for parameter types with wildcards for JarInfer #1107

Merged
merged 4 commits into from
Dec 22, 2024

Conversation

msridhar
Copy link
Collaborator

Partially addresses #1096

Converts wildcard types stored in bytecodes to the corresponding source-level types. We will add this support to the library models code in a follow up.

Copy link

codecov bot commented Dec 21, 2024

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 88.12%. Comparing base (2754c45) to head (d27f267).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...llaway/jarinfer/DefinitelyDerefedParamsDriver.java 77.77% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1107   +/-   ##
=========================================
  Coverage     88.11%   88.12%           
- Complexity     2212     2221    +9     
=========================================
  Files            84       84           
  Lines          7188     7197    +9     
  Branches       1428     1430    +2     
=========================================
+ Hits           6334     6342    +8     
- Misses          429      430    +1     
  Partials        425      425           

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

@msridhar msridhar enabled auto-merge (squash) December 22, 2024 18:33
@msridhar msridhar merged commit f2bafb2 into master Dec 22, 2024
10 of 11 checks passed
@msridhar msridhar deleted the 2024-12-21-jarinfer-wildcards branch December 22, 2024 18:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants