Open grahamwhaley opened 1 year ago
I should be able to get to this in about 2 weeks, I hope that is ok. I am keen to support a great project. I assume you would like me to merge into master and then start a PR? R
Hi @robaol . Thanks for the reply! No rush here on my side - just if and when you are ready or want to :-)
I don't know how familiar you are with github PR flow and processes. Each project and user probably has their own preferences, but if it were me I'd keep those patches on your OtherLCDs
branch, and send the PR from that branch. Then when the PR gets merged into my master
, you can pull updates back into your master
. That is, you might want to use your forks master
branch to track my master
branch. Make sense?
That flow also helps for if you have multiple PRs in flight for the same project - they can each live on their own branch, and your master
remains 'clean', iyswim.
Apologies if you know all that already. If not, github have a number of helpful documents on making and submitting PRs etc.
A couple of thoughts/requests for your PR:
git log
shows a nice trail of edits etc. A git log
in the DPSham repo will show examples of my commits for instance. You can edit the last commit in your branch with git commit --amend
, which is very handy in such cases :-)Signed-off-by
to your commit (git commit -s
). That's not strictly necessary for this project (yet) as I've not defined what it means, but in theory let's say it means you hand over the rights to those changes to this project
). It's not like I'm expecting the project to gain a huge community or commercial/industrial usage though, so I'm not that bothered right now ;-)Thanks!
(I used to do this a lot commercially in the past, so mostly it is 2nd nature for me, but I might have to refresh my memory a little ;-) ).
Hi Graham, Thanks for your tips and suggestions. I will follow them as best I can but my work is rather manic 2nd half year amd so it is likely going to be December before I can give this the concentration it needs to do it properly. Anyway, I have forgotten it and do intend to make the PR. Best regards Rob
Whilst perusing the forks of DSPham (something I do now and then), I spotted that @robaol had added support for a different I2C LCD in their repo (nice!). I'd happily consider adding this code if a suitable pull request were submitted @robaol :-)
For reference, the patch adds support using the same ArduinoMenu system, supporting the I2C LCD code from mathertel.