Skip to content

fix: comment improvements - #1212

Merged
jpoehnelt merged 3 commits into
mainfrom
fix/comment-improvements
Sep 15, 2025
Merged

fix: comment improvements#1212
jpoehnelt merged 3 commits into
mainfrom
fix/comment-improvements

Conversation

@jpoehnelt

Copy link
Copy Markdown

No description provided.

CopilotAI left a comment

Copy link
Copy Markdown

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 improves comments throughout the codebase by fixing formatting issues, correcting spelling and punctuation, enhancing clarity, and updating documentation to use proper terminology. The changes focus on making documentation more consistent and professional.

  • Fix commented-out copyright header in test file
  • Improve JSDoc comment formatting and punctuation
  • Update terminology from "Team Drives" to "shared drives"

Reviewed Changes

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

Show a summary per file
FileDescription
slides/snippets/test/test_slides_refresh_sheets_chart.jsUncommented the copyright header
slides/snippets/slides_create_slide.jsFixed return variable bug and improved comment formatting
slides/snippets/slides_create_sheets_chart.jsImproved comment line wrapping
slides/snippets/slides_create_image.jsBetter comment line breaks
sheets/snippets/*.jsEnhanced JSDoc comments with proper return types and descriptions
forms/snippets/anyone_with_link_responder.jsImproved console message formatting and clarity
drive/snippets/test/*.jsUpdated "Team Drives" to "shared drives" terminology
drive/snippets/drive_v*/file_snippets/*.jsEnhanced function documentation and parameter descriptions
chat//.jsImproved comment formatting and line wrapping
apps-script/execute/index.jsBetter comment structure and clarity
Comments suppressed due to low confidence (1)

slides/snippets/slides_create_slide.js:1

  • The variable res is not defined. Should return result which is the actual response from the API call.
/**

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment threaddrive/snippets/drive_v3/file_snippets/download_file.js
Comment threaddrive/snippets/drive_v3/file_snippets/download_file.js Outdated

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 48 out of 48 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

slides/snippets/slides_create_slide.js:1

  • The variable res is undefined. This should return result which contains the API response data.
/**

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jpoehnelt
jpoehnelt merged commit 583d3c9 into mainSep 15, 2025
12 checks passed
@jpoehnelt
jpoehnelt deleted the fix/comment-improvements branch September 15, 2025 19:42
Sign up for freeto 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

@jpoehnelt