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

Part 4 | Import organization #3804

Conversation

4everTheOne
Copy link
Contributor

Hello, this PR is part of series of 5 PR that will break down the import organization into smaller PR.

(Continuation of #3803)

In this PR, we organize the imports for:

@codecov
Copy link

codecov bot commented Dec 20, 2022

Codecov Report

Merging #3804 (4fe5f0e) into master (147ffcd) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##              master     #3804     +/-   ##
=============================================
  Coverage     57.560%   57.560%             
+ Complexity      5482      2182   -3300     
=============================================
  Files            638       638             
  Lines          33907     33907             
  Branches        5863      5863             
=============================================
  Hits           19517     19517             
  Misses         12308     12308             
  Partials        2082      2082             
Flag Coverage Δ
AlsoSlowTests 57.560% <ø> (ø)
javaparser-core 50.324% <ø> (ø)
javaparser-symbol-solver 36.579% <ø> (ø)
jdk-10 57.556% <ø> (+0.005%) ⬆️
jdk-11 57.556% <ø> (ø)
jdk-12 57.550% <ø> (-0.006%) ⬇️
jdk-13 57.556% <ø> (ø)
jdk-14 57.550% <ø> (-0.006%) ⬇️
jdk-15 57.556% <ø> (+0.020%) ⬆️
jdk-16 57.556% <ø> (ø)
jdk-8 57.554% <ø> (ø)
jdk-9 57.553% <ø> (+0.005%) ⬆️
macos-latest 57.551% <ø> (+0.004%) ⬆️
ubuntu-latest 57.545% <ø> (ø)
windows-latest 57.542% <ø> (ø)

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

Impacted Files Coverage Δ
...thub/javaparser/symbolsolver/JavaSymbolSolver.java 59.459% <ø> (ø)
...r/symbolsolver/javaparsermodel/FailureHandler.java 42.857% <ø> (ø)
...symbolsolver/javaparsermodel/JavaParserFacade.java 74.242% <ø> (ø)
...ymbolsolver/javaparsermodel/JavaParserFactory.java 87.777% <ø> (ø)
...er/symbolsolver/javaparsermodel/TypeExtractor.java 74.308% <ø> (ø)
...arsermodel/contexts/AbstractJavaParserContext.java 80.468% <ø> (ø)
...er/javaparsermodel/contexts/BinaryExprContext.java 87.500% <ø> (ø)
...ver/javaparsermodel/contexts/BlockStmtContext.java 89.655% <ø> (ø)
...l/contexts/ClassOrInterfaceDeclarationContext.java 88.461% <ø> (ø)
...vaparsermodel/contexts/CompilationUnitContext.java 80.254% <ø> (ø)
... and 67 more

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 147ffcd...4fe5f0e. Read the comment docs.

@jlerbsc jlerbsc merged commit 708382b into javaparser:master Dec 22, 2022
@jlerbsc jlerbsc added this to the Next release milestone Dec 22, 2022
@jlerbsc jlerbsc added the PR: Changed A PR that changes implementation without changing behaviour (e.g. performance) label Dec 22, 2022
@4everTheOne 4everTheOne deleted the improvement/organized-imports-solver-core branch December 22, 2022 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Changed A PR that changes implementation without changing behaviour (e.g. performance)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants