baneja opened a new pull request, #131:
URL: https://github.com/apache/maven-assembly-plugin/pull/131

   ### I have refactored the code using following techniques in the mentioned 
classes:
   
   1. Refactoring name: **Extract Method**
   Location: 
src/main/java/org/apache/maven/plugins/assembly/utils/AssemblyFileUtils.java
   Line 69
   
   2. Refactoring name: **Introduce explaining variable**
   Location: 
src/main/java/org/apache/maven/plugins/assembly/mojos/SingleAssemblyMojo.java

   Line 71
   
   3. Refactoring name: **Rename Method**
   Location: 
src/main/java/org/apache/maven/plugins/assembly/format/ReaderFormatter.java
   Line 95
   
   4. Refactoring name: **Replace conditional with Polymorphism**
   Location: 
src/main/java/org/apache/maven/plugins/assembly/io/DefaultMessageHolder.java
   Line 514
   
   5. Refactoring name: **Extract Class**
   Location: 

src/main/java/org/apache/maven/plugins/assembly/archive/archiver/PrefixedFileSet.java
   Line 46 and 82
   
   **Made new classes**

   RootPrefix.java: 
src/main/java/org/apache/maven/plugins/assembly/archive/archiver/RootPrefix.java
   Prefix Prefixed.java: 
src/main/java/org/apache/maven/plugins/assembly/archive/archiver/PrefixedPrefix.java
   
   6. Refactoring name: **Push Down Method**
   Location: 
FileLocation.java: 
src/main/java/org/apache/maven/plugins/assembly/io/FileLocation.java
   Line 145
   URLLocation.java: 
src/main/java/org/apache/maven/plugins/assembly/io/URLLocation.java
   Line 81
   
   
   
   


-- 
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: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to