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

[AutoPR datamigration/resource-manager/v2018_07_15_preview] Fix bad discriminator value on ConnectionInfo for node/js SDK #156

Conversation

openapi-sdkautomation[bot]
Copy link

mitchdenny and others added 19 commits August 13, 2019 21:15
* Explicit checkout on build step to test.

* Adding checkout steps to test and analyze.
[pull] master from Azure:master
Adds a flag to the pipeline which controls whether it is a track 1 or track 2 build.
* Formatting changes in EventHubAsyncProducer.

* Adding EventHubClient, EventHubConsumer, and EventHubProducer.

* Exposing EventHubClient creation in EventHubClientBuilder.

* EventHubClient, Consumer and Producer implements Closeable.

* Fixing sample by removing event hub instance from namespace connection string.

* Remove unneeded sample in EventHubClientBuilder.

* Add EventHubClient to builder annotation.

* Update EventHubClientBuilder samples. Remove unneeded ones.

* Update samples in EventHubClientBuilder.

* Fixing links to EventHubAsyncProducer samples.

* Adding EventHubProducer code samples.

* Update from Iterable to IterableResponse.

* Make test contents package-private.

* Adding tests for EventHubProducer.

* Simplifying creation of EventHubAsyncProducer

* Select correct retryDuration when constructing EventHubProducer.

* Adding EventHubProducer tests.

* Rename EventHubClientIntegrationTest -> EventHubAsyncClientIntegrationTests

* Add integration tests for EventHubClient.

* Make EventHubConsumer methods public
* Split the eventhubs client and data pipelines.

* Removed redundant modules section.

* Specified artifacts for data pipeline.
* Seeing if we can speed up spotbugs by not forking the JVM for each module
* remove java code isImple check but move to suppression and add only check for public class for external Dependency check

* fixes

* Update eng/code-quality-reports/src/main/resources/checkstyle/checkstyle-suppressions.xml

suggest changes on comments

Co-Authored-By: Connie Yau <conniey@microsoft.com>
* Move Storage to official swagger files

There are no (meaningful) code changes except the attribute in
BlobHierarchyListSegment that I can't trace to the swagger

* Upgrade blobs swagger to 2019-02-02

Note that additional work will have to be done to patch our higher level code to
pass new parameters to existing APIs

* Fixes to Swagger generation and updating to use new protocol layer

* Fixed appendBlock content-type, added SpotBug exclusions for autogen code

* Update README.md

* Fixed incorrect exclude
Fix bad discriminator value on ConnectionInfo which was breaking node/js SDK generation
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.

9 participants