Skip to content

Conversation

danuw
Copy link
Collaborator

@danuw danuw commented Jul 14, 2025

…linked to PR #622

Pull Request

Issue Number: #622

Summary

Addresses syntax errors spotted in #622 and others

Changes

Checklist

  • Local Tests Passing?
  • CICD and Pipeline Tests Passing?
  • Added any new Tests?
  • Documentation Updates Made?
  • Are there any API Changes? If yes, please describe below.
  • This is not a breaking change. If it is, please describe it below.

Are there API Changes?

If yes, what are the expected API Changes? Please link to an API-Comparison
workflow with the API Diff.

Is this a breaking change?

If yes, what workflow does this break?

Anything else?

Other comments, collaborators, etc.

Please follow
GitHub's suggested syntax
to link Pull Requests to Issues via keywords

This PR Closes #622

@danuw danuw requested a review from vaughanknight as a code owner July 14, 2025 19:23
@danuw danuw requested review from Copilot and vaughanknight and removed request for vaughanknight July 14, 2025 19:23
@danuw danuw added the v1.9 label Jul 14, 2025
Copy link
Contributor

@Copilot 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

Addresses spelling and grammatical errors in both the SECURITY documentation and the project README to improve clarity and prevent build issues.

  • Fixed multiple typos and improved phrasing in SECURITY.md
  • Corrected grammatical constructions and updated screenshot alt text in README.md

Reviewed Changes

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

File Description
SECURITY.md Corrected typos for consistency (e.g., “maintaned” → “maintained”), standardized phrasing, and fixed header formatting
README.md Improved sentence structure in the intro, fixed verb agreement, and updated screenshot alt text

@danuw danuw moved this to In Progress in Carbon Aware SDK Jul 14, 2025
danuw and others added 2 commits July 15, 2025 08:22
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Dan Benitah <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Dan Benitah <[email protected]>
@danuw danuw merged commit f233206 into dev Jul 15, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Carbon Aware SDK Jul 15, 2025
@danuw danuw deleted the docs/typos-622-250714 branch July 15, 2025 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants