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.

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

Removed Unused Snippets from Code [Correct PR] #22859

Merged
merged 1 commit into from
Jul 26, 2021

Conversation

AhmedLeithy
Copy link
Member

Excuse the mess with the previous PR, I have closed it. The following four unused snippets were removed from the code, as they are not referenced in any Markdown documents.

translation: Snippet:MultipleInputs
translation: Snippet:OperationsHistory
translation: Snippet:PollIndividualDocuments
translation: Snippet:StartTranslationWithAzureBlobAsync

This is PR is in reference to issue #22130 .

@AhmedLeithy AhmedLeithy requested a review from maririos as a code owner July 25, 2021 13:55
@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Jul 25, 2021
@ghost
Copy link

ghost commented Jul 25, 2021

Thank you for your contribution @AhmedLeithy! We will review the pull request and get back to you soon.

@ghost ghost added the Community Contribution Community members are working on the issue label Jul 25, 2021
@maririos maririos added Client This issue points to a problem in the data-plane of the library. Cognitive - Translator and removed Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Jul 26, 2021
@maririos
Copy link
Member

Thank you @AhmedLeithy !!

@maririos maririos merged commit e065745 into Azure:main Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Translator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants