-
Notifications
You must be signed in to change notification settings - Fork 76
Archive this repository #368
Comments
Not sure if we should keep it just to point to to JP. We could also renamed it to |
I am currently using older versions of JavaParser and JavaSymbolSolver. If i want to migrate to the new merged version, which maven dependency do i need to install. These two together seem to work for now:
JavaParser-Core does not come with the Symbol Solver yet, does it? When is it expected to come in one dependency? By the way, putting these projects together is a great step forward! Looking forward to all the upcomming changes. |
Yes, from now on just put the same version for both the modules and you are done. I think you can also just include the javaparser-symbol-solver-core and it will depend on javaparser-core, so it will be included automatically. At least, I would expect it to work in this way |
You are right. Can confirm that only including Just read a few minutes ago on Twitter that the symbol solver has been put into the java parser repo, but only including the javaparser-core dependency in my project didnt include the symbol solver part. Is this going to be the long term plan, because I think most people arrive at the JavaParser repo and use the Maven dependency text from there, not realizing that the Symbol Solver doesnt come with it. |
Does, it is not the plan. We will have several modules under the JavaParser repository. Some people will be happy using just the javaparser-core module while others will need javaparser-symbol-solver-core. They will need to figure out they need to add a separate module. We will try to make it clear (and we are open to suggestions on this) but we expect the users to read the Readme (that we should update) |
I see, that makes sense too. I think in general having a nice looking and well structured readme is always important, not only for providing useful information, but also for having a professional appearances for new users. |
I think it is now possible to archive projects so I will rename this issue |
I've transfered the remaining issues, so here we go! |
When enough stuff has been moved to the javaparser repo.
The text was updated successfully, but these errors were encountered: