Haven’t made any contributions since Openfire was in SVN.
Is there a contributors or developers guide for best practices on the new Github repo?
Need to make a few changes to the Openfire code base with regards to the Monitoring plugin etc. is best practise to create pull requests for every change, for major ones, or do all devs here commit directly to master?
Usually you put your work on a feature or bug fix in its own branch. Then commit your changes to your branch.
If you are done, you can make a pull request against the igniterealtime/Openfire master (or a branch, but for now, all have made their PR against master).
Seems to be good practice to rebase your branch to master and squash it into a single commit before the PR, but I am not sure about. It probably depends on what you are doing. Several logical commits, which are only corrections or test/development commits should probably be squashed. If there’s more work, keep them separated, I’d say.
I think nobody has committed directly to master, except Dele (for Video stuff only). PRs are a nice thing to see changes.
So for example on ticket OF-672 I need to make a 5 line change to a properties file. As an Openfire dev with write access (I think ) do I clone, branch, submit a pull request and then approve my own pull request to make such a minor change?
Obviously we want to really avoid mistakes and rubbish ending up in the official codebase but is this required for minor changes?