grass icon indicating copy to clipboard operation
grass copied to clipboard

Fix partial i18n of line

Open slichtzzz opened this issue 2 years ago • 3 comments

Fix partial i18n of line

slichtzzz avatar Mar 23 '23 08:03 slichtzzz

The spelling fixes are good, but please add more in a separate PR, not here, we try to limit the scope of PRs to make it easier to review PRs now and the code changes in the future.

wenzeslaus avatar Mar 24 '23 15:03 wenzeslaus

Hi! I'm sorry. I'm trying to put the commit in a new pull request, but despite all my efforts all the stuff is going back to the same old pull request. Apparently, I'm doing something wrong, but don't see how to fix it…

slichtzzz avatar Mar 27 '23 07:03 slichtzzz

...put the commit in a new pull request, but despite all my efforts all the stuff is going back to the same old pull request...

On your local machine, you interact with branches, the commits go to branches, then you open one PR per branch.

Your branch for this PR is main which is not great, but not a deal breaker. The best next step depends on whether you are using a local Git repo or doing things in the GitHub web interface only. Trying both ways may give you some insights. On web, you are trying to modify your fork, try to modify this repo and it should lead you the right way.

In any case, check the CONTRIBUTING.md file and some basic tutorials. I think it is a confusing process the first time around, but I don't have any remedy for that.

wenzeslaus avatar Mar 27 '23 17:03 wenzeslaus

This will probably be closed by #4052 once merged. That other PR adresses all of these issues in Python code.

echoix avatar Jul 19 '24 17:07 echoix