Skip to content

invalid non ascii character was causing problems #7102

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
Merged
mmyyrroonn merged 1 commit into from
Sep 3, 2019
Merged

invalid non ascii character was causing problems #7102

mmyyrroonn merged 1 commit into from
Sep 3, 2019

Conversation

@zikalino
Copy link

@zikalino zikalino commented Sep 3, 2019

non-ascii character was causing problems with python sdk tools so changing it to proper one

@zikalino zikalino requested a review from jaredmoo as a code owner September 3, 2019 03:50
@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Sep 3, 2019 •
edited
Loading

In Testing, Please Ignore

[Logs] (Generated from c9732de, Iteration 1)

Warning .NET: test-repo-billy/azure-sdk-for-net [Logs] [Diff]
  • No packages generated.
Failed Python: test-repo-billy/azure-sdk-for-python [Logs] [Diff]
Failed Java: test-repo-billy/azure-sdk-for-java [Logs] [Diff]
Warning Go: test-repo-billy/azure-sdk-for-go [Logs] [Diff]
  • Warning preview/sql/mgmt/2015-05-01-preview [Logs]
  • Warning preview/sql/mgmt/2017-03-01-preview [Logs]
  • Warning preview/sql/mgmt/2017-10-01-preview [Logs]
  • Warning sql/mgmt/2014-04-01 [Logs]
Succeeded JavaScript: test-repo-billy/azure-sdk-for-js [Logs] [Diff]
Succeeded Ruby: test-repo-billy/azure-sdk-for-ruby [Logs] [Diff]

@AutorestCI
Copy link

AutorestCI commented Sep 3, 2019 •
edited
Loading

Automation for azure-sdk-for-python

A PR has been created for you:
Azure/azure-sdk-for-python#7041

@AutorestCI
Copy link

AutorestCI commented Sep 3, 2019 •
edited
Loading

Automation for azure-sdk-for-go

A PR has been created for you:
Azure/azure-sdk-for-go#5680

@azuresdkci
Copy link
Contributor

azuresdkci commented Sep 3, 2019

Can one of the admins verify this patch?

@mmyyrroonn mmyyrroonn merged commit 7d870c5 into master Sep 3, 2019
This was referenced Sep 3, 2019
Merged
Merged
@JackTn JackTn deleted the removed-invalid-non-ascii-character branch August 9, 2021 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jaredmoo jaredmoo Awaiting requested review from jaredmoo

@michaeljqzq michaeljqzq Awaiting requested review from michaeljqzq

1 more reviewer

@mmyyrroonn mmyyrroonn mmyyrroonn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@michaeljqzq michaeljqzq

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@zikalino @AutorestCI @azuresdkci @mmyyrroonn @michaeljqzq