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

Updating the Datafactory SDK from swagger repo, preparing release #6971

Merged
merged 7 commits into from
Jul 19, 2019
Merged

Updating the Datafactory SDK from swagger repo, preparing release #6971

merged 7 commits into from
Jul 19, 2019

Conversation

hvermis
Copy link
Contributor

@hvermis hvermis commented Jul 18, 2019

No description provided.

@wenbof-zz
Copy link

@hvermis there is one change(sent out for more than one week now) not checked in which needs to be in the SDK by end of this month.

@hvermis
Copy link
Contributor Author

hvermis commented Jul 19, 2019

@wenbof Let's do that in a separate PR to minimize the risk of not making it.

@wenbof-zz
Copy link

wenbof-zz commented Jul 19, 2019

@hvermis Hopefully we can make that by end of this month. it is a hard date. Thanks. if you think that there is low possibility that we can make it, please help wait for this change. thanks.

@erich-wang erich-wang self-assigned this Jul 19, 2019
@erich-wang
Copy link
Member

@hvermis , could you please share swagger spec PR link related to this change? I need to understand what's change in the spec PR and whether it is merged?

@hvermis
Copy link
Contributor Author

hvermis commented Jul 19, 2019

@erich-wang I generated the .Net code from the public repo url, ADF readme file. So this code corresponds exactly to whatever is merged in swagger repo. Here are the PRs in swagger repo:
Azure/azure-rest-api-specs#6631
Azure/azure-rest-api-specs#6548
Azure/azure-rest-api-specs#6607
Azure/azure-rest-api-specs#6568

@erich-wang erich-wang merged commit 0795580 into Azure:master Jul 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants