-
Notifications
You must be signed in to change notification settings - Fork 492
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
Dependency Housekeeping and how developers will benefit from it #5288
Comments
Please note that #5289 is not solving all of the aspects of this story. Those still need to be discussed. Ping to @matthew-a-dunlap and @scolapasta. |
Over at http://irclog.iq.harvard.edu/dataverse/2018-11-27#i_80014 @poikilotherm let me know that he removed the commit that he believes was causing the exception @kcondon found at #5274 (comment) reported as "I'm able to build and use somewhat the resulting war file but it throws a stack trace about a missing method when I load the homepage and an exception (not logged) when clicking on a dataset card to view the dataset page." @poikilotherm indicated that the change he made was the following: "I removed the commit that removed the TrueZIP stuff and removed the commit that was changing the AWS dependency" I moved this issue to code review at https://waffle.io/IQSS/dataverse |
@poikilotherm I hate to say this, but I'd love for you to adjust a bit more to our process. "Small chunks" is something you'll hear us say all the time. "Incremental value." Agile, kanban, whatever. Every team has their own spin. First of all, thank you very much for this pull request! We are close to a release, I think. Only one more issue to go. After we cut a release, we'll want to indicate which issues were addressed in that release. I would like this issue to remain closed and have a clear title of what was delivered. What I don't want is the confusion of release note pointing to this issue with a status of "open" and a check list of what's done and what's not done. "Definition of done" is something you'll hear us say a lot too. It's fine to create "epic" issues for organizational purposes but before an issue goes into dev, code review, and especially QA, the "definition of done" should be clear enough that it can be closed. I hope this is making sense. Something else to think about is that buried in the pull request that was just merged is a new ability to use graphviz in the guides. However, there is no issue representing this improvement, nothing to link to from release notes that describes this new feature. Dataverse installations who build their own guides may or may not notice warnings after the next release. This would have been a good issue to split into a "small chunk". I'm sure it's annoying to have to adjust to the process of this or that team and probably we should document more of the process so that people are surprised by how we operate. I can't emphasize enough how much I appreciate the energy you've put into Dataverse already. I'm just trying to smooth things out in terms of how issues and pull requests make their way through QA. To summarize:
I hope this helps. 😄 |
Sorry to hear that @pdurbin . I am really trying to stick to your flow, but it seems I still need to adopt IQSS sense of "how small is small". I am fine with leaving this closed if that is more inline with your procedure. I will just open a successor issue then, so the thoughts and ideas above don't get lost. Would it make you feel better, if I open a separate issue for that Graphviz stuff plus make a new PR adding some stuff to release notes? |
@poikilotherm thanks for opening #5360 as a successor issue! |
This issue is a story from or a blocker to #5292.
This issue is related to #4260, #5274, #3921, #4172 and more.
Please feel addressed @matthew-a-dunlap and @scolapasta 😉
While working on #4172 I discovered the problems noted down in #5274, which are in turn a left-over of #3921. This lead me to the impression, that the current
pom.xml
could use some love in terms of dependency management, hopefully avoiding more problems down the road. More and more deps keep kicking in while functionality is added to Dataverse. (As @pdurbin said on IRC: Dataverse 4.0 was around 45MB, 4.9.4 is at 146 MB)I made some first steps primarily to address #5274, but this might be a could starting point for more work in this. Reducing WAR size, find wrong imports, update dependencies, address convergence issues and more...
Done so far
(mostly for the resolution of #5274)local_lib
JARs to make Maven happierThings open (and will stay open till we are on Glassfish 5+, see #5274)
Things to consider
local_lib
into a GitHub based "poor mans" Maven repo.mvn enforcer:enforce
mvn dependency:analyze
The text was updated successfully, but these errors were encountered: