Skip to content

chore: updated all servers docstrings for better summary#191

Merged
rileyok-ons merged 8 commits into
190-documentation-tidyup-for-v110from
190-documentation-tidyup-for-v110-servers
Jun 29, 2026
Merged

chore: updated all servers docstrings for better summary#191
rileyok-ons merged 8 commits into
190-documentation-tidyup-for-v110from
190-documentation-tidyup-for-v110-servers

Conversation

@rileyok-ons

Copy link
Copy Markdown
Contributor

📌Add server docstring updates

Please complete all sections

✨ Summary

📜 Changes Introduced

  • Feature implementation (feat:) / bug fix (fix:) / refactoring (chore:) / documentation (docs:) / testing (test:)
  • Updates to tests and/or documentation
  • Terraform changes (if applicable)

✅ Checklist

Please confirm you've completed these checks before requesting a review.

  • Code passes linting with Ruff
  • Security checks pass using Bandit
  • API and Unit tests are written and pass using pytest
  • Terraform files (if applicable) follow best practices and have been validated (terraform fmt & terraform validate)
  • DocStrings follow Google-style and are added as per Pylint recommendations
  • Documentation has been updated if needed

🔍 How to Test

@rileyok-ons rileyok-ons requested a review from a team as a code owner June 23, 2026 16:19
@github-actions github-actions Bot added the chore label Jun 23, 2026

@lukeroantreeONS lukeroantreeONS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work @rileyok-ons , happy to approve

@rileyok-ons rileyok-ons merged commit 8dba97c into 190-documentation-tidyup-for-v110 Jun 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants