aherbert commented on pull request #114:
URL: https://github.com/apache/commons-csv/pull/114#issuecomment-702966394


   Thanks for the PR. You have removed some public API methods. This is not 
allowed and would fail binary compatibility tests. I have enabled these on the 
CI environments. If you rebase on master the errors should appear in the checks.
   
   You can run:
   ```
   mvn
   ```
   
   on the project to see the full checks on the code. 
   
   The `mvn clirr:check` goal will fail and indicate the public methods that 
have been removed. These should be supported in addition to the new methods 
with the new duplicate header names enum.
   


----------------------------------------------------------------
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.

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


Reply via email to