Skip to content

Update refactoring-python-code.md #8385

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.

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

Merged
merged 1 commit into from
Aug 26, 2022
Merged

Conversation

ijoosong
Copy link
Contributor

Removing "Extract Method" as it is deprecated

Removing "Extract Method" as it is deprecated
@prmerger-automator
Copy link
Contributor

@ijoosong : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@opbld33
Copy link

opbld33 commented Aug 22, 2022

Docs Build status updates of commit 816e6b9:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/python/refactoring-python-code.md ⚠️Warning Details

docs/python/refactoring-python-code.md

  • Line 22, Column 3: [Warning: bookmark-not-found] Cannot find bookmark '#extract-method' in 'python/refactoring-python-code.md'.

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Aug 22, 2022

@rjmolyneaux

  • Can you review this PR?

Note that the validation warning will need to be cleared/addressed before this can be merged. thanks.

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Aug 22, 2022
@j-martens j-martens merged commit a5c2131 into MicrosoftDocs:main Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants