cquoss edited a comment on pull request #110: URL: https://github.com/apache/maven-scm/pull/110#issuecomment-832017583
Thanks. That is very much appreciated. I pushed my work in progress so you see what i am heading at. And can stop me in my tracks if i run astray in your opinion. Two more things to think about: 1. I change the changelog command implementation to: provider->request / command->parameters. In some cases (svnexe f.i.) request has been handed down to command. I changed that back to parameters. 2. Why do i have to install svn.exe to make the svnexe unit tests run, but no other provider needs an installation for that (OK, i do not know about git.exe since this was installed already)? Latest commit compiles again and unit tests are without fault or error. I will check my changes and remove any unneccessary ones. Then the commits can be squashed to one. Maybe no ned to merge SCM-947 and SCM-948 but integrate them both one after the other, SCM-948 first. But SCM-947 might need rework. Haven't checked yet. Should this below be tackled, too? [WARNING] The project org.apache.maven.scm:maven-scm:pom:1.11.3-SNAPSHOT uses prerequisites which is only intended for maven-plugin projects but not for non maven-plugin projects. For such purposes you should use the maven-enforcer-plugin. See https://maven.apache.org/enforcer/enforcer-rules/requireMavenVersion.html -- 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]
