Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fill in todos for powerrename.md #30274

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

fill in todos for powerrename.md #30274

wants to merge 3 commits into from
+286 −9

Conversation

megan-boyd
Copy link

#17951

Summary of the Pull Request

add in relevant documentation to powerrename.md.

alvinashcraft
alvinashcraft reviewed Dec 15, 2023
View reviewed changes
Copy link
Contributor

@alvinashcraft alvinashcraft left a comment

Choose a reason for hiding this comment

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

Is the one Usage section still in progress?

doc/devdocs/modules/powerrename.md Outdated Show resolved Hide resolved
doc/devdocs/modules/powerrename.md

## Usage

Provide information on how to compile and use this utility. Include any dependencies or specific build instructions if necessary.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this line still a TODO?

@Jay-o-Way Jay-o-Way requested a review from crutkas December 21, 2023 13:30
@jaimecbernardo jaimecbernardo added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jan 23, 2024
megan-boyd and others added 2 commits February 2, 2024 14:48
@megan-boyd @alvinashcraft
Update doc/devdocs/modules/powerrename.md
3e445b9 
Co-authored-by: Alvin Ashcraft <[email protected]>
@megan-boyd
Update powerrename.md
57207db 
Removed extraTODO in readme file
@jaimecbernardo
Copy link
Collaborator

Hi @megan-boyd , Thanks for opening the PR. We need the CLA signed before being able to consider this PR.

@crutkas
Copy link
Member

crutkas commented Jul 17, 2024

@megan-boyd we can't merge this until you agree to the CLA. We'll have to close the PR if you don't.

@megan-boyd
Copy link
Author

@microsoft-github-policy-service agree

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alvinashcraft alvinashcraft alvinashcraft left review comments

@crutkas crutkas Awaiting requested review from crutkas

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Needs-Team-Response An issue author responded so the team needs to follow up Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams
Projects
5. Needs Attention
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@megan-boyd @jaimecbernardo @crutkas @alvinashcraft