Skip to content

Conversation

@nsheff
Copy link
Member

@nsheff nsheff commented Nov 6, 2025

Updated docstrings throughout the pypiper codebase from reStructuredText style (:param, :return, :raise) to Google-style format (Args:, Returns:, Raises:) for improved readability and consistency.

Modified files:

  • pypiper/manager.py: Converted PipelineManager class and all 27+ methods
  • pypiper/pipeline.py: Converted Pipeline class and all methods
  • pypiper/utils.py: Converted 28+ utility functions
  • pypiper/stage.py: Converted Stage class and methods
  • pypiper/ngstk.py: Converted NGSTk class and 78+ methods
  • pypiper/exceptions.py: Converted UnknownPipelineStageError
  • pypiper/folder_context.py: Converted FolderContext class

All conversions maintain original descriptions and documentation details while following Google-style documentation standards.

donaldcampbelljr and others added 7 commits April 19, 2024 09:54
Updated docstrings throughout the pypiper codebase from reStructuredText
style (:param, :return, :raise) to Google-style format (Args:, Returns:,
Raises:) for improved readability and consistency.

Modified files:
- pypiper/manager.py: Converted PipelineManager class and all 27+ methods
- pypiper/pipeline.py: Converted Pipeline class and all methods
- pypiper/utils.py: Converted 28+ utility functions
- pypiper/stage.py: Converted Stage class and methods
- pypiper/ngstk.py: Converted NGSTk class and 78+ methods
- pypiper/exceptions.py: Converted UnknownPipelineStageError
- pypiper/folder_context.py: Converted FolderContext class

All conversions maintain original descriptions and documentation details
while following Google-style documentation standards.
@nsheff nsheff changed the base branch from master to dev November 6, 2025 00:05
@nsheff nsheff requested a review from Copilot November 6, 2025 01:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the docstring format across the pypiper codebase by converting from reStructuredText (reST) format to Google-style docstrings. The changes affect multiple files including utils.py, stage.py, pipeline.py, ngstk.py, manager.py, folder_context.py, and exceptions.py.

Key changes:

  • Converted parameter documentation from :param type name: to Args: section with name (type): format
  • Converted return documentation from :return type: to Returns: section
  • Converted exception documentation from :raise Exception: to Raises: section
  • Moved opening docstring quotes to same line as the summary sentence

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pypiper/utils.py Converted all function docstrings from reST to Google style
pypiper/stage.py Updated class and method docstrings to Google style format
pypiper/pipeline.py Modernized all pipeline-related docstrings
pypiper/ngstk.py Converted NGSTk class and method docstrings
pypiper/manager.py Updated PipelineManager docstrings to Google style
pypiper/folder_context.py Converted context manager docstrings
pypiper/exceptions.py Updated exception class docstrings

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

nsheff and others added 2 commits November 5, 2025 20:13
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nsheff nsheff merged commit 0b4a56f into dev Nov 6, 2025
3 checks passed
@nsheff nsheff deleted the claude/update-docstrings-google-style-011CUqZJhqzgD9t3NY6VSpFx branch November 6, 2025 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants