epoyraz / mztab

Automatically exported from code.google.com/p/mztab
0 stars 0 forks source link

Code Review Request for jmztab/branches/jmztab-modular #8

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:
Modularization of the monolithic maven project. The following modules were 
added:

jmztab-model
jmztab-util
jmztab-cli
jmztab-gui
jmztab-converter-pride

The artifact Ids have been changed for now for disambiguation reasons, e.g. for 
the jmztab-model module, the artifact id is "jmztab-modular-model", where 
"jmztab-modular" is the artifact id of the parent pom.xml of the multi-module 
project. 
The artifact Ids are preliminary proposals. 

When reviewing my code changes, please focus on:
I checked that all test cases still run, however, please stay alert for 
possible issues. The main benefit of this modularization is that users do not 
have to include all dependencies since all modules have clear and minimal 
dependencies. This allows selective inclusion of the cli, gui, and 
converter-pride modules, for those who need them. The model module now only 
contains domain-dependent code. The util module contains the former 
...jmztab.utils package code, expect for the pride converter related code, 
which now resides in its own module.  

After the review, I'll merge this branch into:
jmztab/trunk

Original issue reported on code.google.com by nils.hof...@web.de on 28 Mar 2014 at 2:16