Skip to content

Comments

[AutoPR servicefabric/data-plane] Service Fabric REST API specification version 6.2 RTO#2383

Merged
AutorestCI merged 4 commits intorestapi_auto_servicefabric/data-planefrom
restapi_auto_2855
Apr 20, 2018
Merged

[AutoPR servicefabric/data-plane] Service Fabric REST API specification version 6.2 RTO#2383
AutorestCI merged 4 commits intorestapi_auto_servicefabric/data-planefrom
restapi_auto_2855

Conversation

@AutorestCI
Copy link
Contributor

@AutorestCI
Copy link
Contributor Author

AutorestCI commented Apr 12, 2018

(message created by the CI based on PR content)

Installation instruction

Package azure-servicefabric

You can install the package azure-servicefabric of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_2855#egg=azure-servicefabric&subdirectory=azure-servicefabric"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_2855#egg=azure-servicefabric&subdirectory=azure-servicefabric"

If you have a local clone of this repository, you can also do:

  • git checkout restapi_auto_2855
  • pip install -e ./azure-servicefabric

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_2855
  • pip wheel --no-deps ./azure-servicefabric

Direct download

Your files can be directly downloaded here:

@samedder
Copy link
Contributor

I think this is okay except @lmazuel there's an error in 2.7 where it looks like it's trying to use 3+ syntax:

__import__(modname)

E File "/home/travis/build/Azure/azure-sdk-for-python/azure-servicefabric/azure/servicefabric/models/test_error_chaos_event_py3.py", line 48
E def init(self, *, time_stamp_utc, reason: str=None, **kwargs) -> None:
E ^
E SyntaxError: invalid syntax

Service Fabric REST API specification version 6.2 RTO
fix the bugs in the specification and examples as surfaced by the example validation
@lmazuel lmazuel force-pushed the restapi_auto_2855 branch from bd3b67f to 2f7910c Compare April 18, 2018 22:45
@lmazuel
Copy link
Member

lmazuel commented Apr 19, 2018

@samedder That's an error from the test framework. ServiceFabric has a class called TestErrorChaosEvent and pytest thinks it's a test class. I will add explicit exceptions to say that no, it's not a test and this should bring back 2.7

remove newlines from the end of the description
@AutorestCI AutorestCI merged commit e477318 into restapi_auto_servicefabric/data-plane Apr 20, 2018
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #2445

@lmazuel lmazuel deleted the restapi_auto_2855 branch April 23, 2018 16:29
lmazuel pushed a commit that referenced this pull request May 16, 2018
* [AutoPR servicefabric/data-plane] Service Fabric REST API specification version 6.2 RTO (#2383)

* Generated from 4cffa20a5783aab5b2d2f0268d922c6b9a6a21ba

Service Fabric REST API specification version 6.2 RTO

* Generated from 9ceed58fbab00d8902f075bda8fa948e27a99c0f

fix the bugs in the specification and examples as surfaced by the example validation

* Generated from dff9557eeb14fa3ed9b82e8dcfa1312a2b67a4ab

example updates

* Generated from 813b94529da22628561e9a804adb53710d08e967

remove newlines from the end of the description

* Avoid pytest to look for tests in folder called 'models'

* Latest Autorest.Python fixes

* ServiceFabric 6.2.0.0 ChangeLog

* Final details
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.

3 participants