Skip to content

Conversation

@ArthPatel1502
Copy link
Contributor

This PR updates the headings in doc/examples/python/script/r.example.md
from ALL CAPS to Title Case as suggested in issue #5239.

This is my first contribution to GRASS GIS.

@github-actions github-actions bot added docs markdown Related to markdown, markdown files labels Dec 2, 2025
@ArthPatel1502 ArthPatel1502 changed the title Docs: convert headings to Title Case in r.example.md (#5239) docs: convert headings to title case in r.example.md Dec 2, 2025
@ArthPatel1502
Copy link
Contributor Author

This is my first contribution to GRASS GIS.
Please let me know if anything needs to be adjusted. 🙂

@ArthPatel1502
Copy link
Contributor Author

“I noticed the ‘Code scanning cannot determine the alerts introduced by this pull request’ warning. Since this PR only touches documentation and not Python code, I assume this is related to the repository’s code scanning configuration and not to this change, but please let me know if I should adjust anything.”

@echoix
Copy link
Member

echoix commented Dec 5, 2025

“I noticed the ‘Code scanning cannot determine the alerts introduced by this pull request’ warning. Since this PR only touches documentation and not Python code, I assume this is related to the repository’s code scanning configuration and not to this change, but please let me know if I should adjust anything.”

Doesn't matter for this kind of PR. And the PR doesn't risk to be too much outdated when one or more unrelated commits are merged to main. It doesn't need to be updated from main after each commit (if done for each PR, it would exponentially increase the time waiting for CI!). We only have the option enabled for us to update PRs without waiting for authors when too old or changes that might impact a PR or merged (even if not a conflict as per say)

@ArthPatel1502
Copy link
Contributor Author

ArthPatel1502 commented Dec 5, 2025 via email

Copy link
Contributor

@cwhite911 cwhite911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @ArthPatel1502

@cwhite911 cwhite911 enabled auto-merge (squash) December 10, 2025 23:56
@cwhite911 cwhite911 merged commit 9870173 into OSGeo:main Dec 13, 2025
22 checks passed
@github-actions github-actions bot added this to the 8.5.0 milestone Dec 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs markdown Related to markdown, markdown files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants