r.latlong: return central pixel values for lat or long

[mankoff]

Attempt to fix issues in #5985

[metzm]

Thanks!

[mankoff]

Good to hear this is approved. I'm fairly new to GRASS development. Should I squash and force-push, update the commit comment, reference #3274, etc.? Anything else from me to do at this point?

[metzm]

Good to hear this is approved. I'm fairly new to GRASS development. Should I squash and force-push, update the commit comment, reference #3274, etc.? Anything else from me to do at this point?

Just press the friendly green "Squash and merge" button and delete your source branch mankoff:latlon. There is no need for any force-push and the commit comment is fine.

If you do not see a friendly green "Squash and merge" button below or if it is greyed out, you probably do not have rights to merge and someone else needs to merge.

[mankoff]

I do not have that green button.

[metzm]

Then I will merge for you, but you need to delete your source branch yourself.

[petrasovaa]

@mankoff Thanks for contributing! Feel free to leave some feedback whether you were able to find contributing guidelines and if so, whether there was something missing that you would find helpful.

[mankoff]

@mankoff Thanks for contributing! Feel free to leave some feedback whether you were able to find contributing guidelines and if so, whether there was something missing that you would find helpful.

I thought the contribute docs were fine. I was just confused because I could not merge and usually I do see the green button. I would have cleaned up the commit message if I had done the final merge, but that is not very important. The relevant info is in the message.

[petrasovaa]

@mankoff Thanks for contributing! Feel free to leave some feedback whether you were able to find contributing guidelines and if so, whether there was something missing that you would find helpful.

I thought the contribute docs were fine. I was just confused because I could not merge and usually I do see the green button. I would have cleaned up the commit message if I had done the final merge, but that is not very important. The relevant info is in the message.

Right, the final merging can only be done by maintainers, but there is unfortunately no way for you to specify the merge commit message (besides explicitly suggesting it in a comment or PR description). The default commit message is by default taken from all the commits in the PR, so it's good to make those commit messages meaningful.