centic9 opened a new pull request #313:
URL: https://github.com/apache/poi/pull/313


   This is a PR for discussing a change to the ooxml-lite packages which I 
would like to commit.
   
   ### Problem
   
   Currently executing `./gradlew jar` takes a very looooong time. 
   
   This is because we run all tests including the long-running 
integration-tests.
   
   The root-cause is that we want to generate a list of all used classes/xsb 
files by running tests. This is then used to build  a minimized 
`poi-ooxml-lite` package which has a reduced size and thus is easier to include 
in some environments.
   
   ### Proposed solution
   
   In order to avoid running all tests whenever building jars, this PR adds the 
list of classes/xsbs to the repository.
   
   This means `./gradlew jar` can run without needing to invoke all tests, 
speeding it up a lot!
   
   Whenever tests are run and lead to new entries to the files, they should be 
checked in along the actual changes to the code.
   
   ### Potential downsides
   
   * When running tests in CI, changes to the new .clazz and .xsb files could 
happen similar to the Java 9-Class-files
   ** => Added steps to revert these two files if necessary
   * When new clazz-files show up, they need to be added to the files.
   ** => Running `./gradlew test` or `./gradlew jenkins` locally will still run 
all tests locally and thus will update the files with new entries as necessary. 
Those can then be committed along with the changed code


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to