@coeing, @GerdZanker and I have developed an binding for the Bosch Smart Home System (https://www.bosch-smarthome.com/de/de). We now have support for quite a number of those devices and all three of us have the binding operational for our Bosch SHCs.
Could you advice on how to proceed? Should we simply open a PR against master of your repo and you will review it there, or are there any other steps that we should follow first.
Opening a pull request is basically the summary of the process. If you want to have it merged with 2.5 you should open a pull request against the 2.5.x branch.
To help the review process it would help if your binding code is in good shape. I glanced at the code and noticed some dead code, dead comments for example. So it’s better to clean it up before. Also have a look through the guidlines page, with information about running the auto format tool and make sure you sign off your commits properly. You can start by running maven with checks not disabled.
Thanks for the quick reply and the link to the guidelines page. We’ll refactor our code so it’ll match the guidelines. We might miss some points as none of us is a Java native speaker But I hope we can figure those points out during the pull request.
One question: What do you mean with “sign off your commits properly”?
We did a feature freeze some weeks ago and used the time to clean up and refactor our code. Plus some fellow devs already tried the plugin at home. They found some minor issues and had some wishes for the next version, but in general it worked quite well already
If you have time, could you have another glance at our code and give your estimation if we are ready to open a pull request? Thanks in advance!