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 notificationhubs/resource-manager] Add missing debugsend and patch operation for NotificationHub resource #2724

Merged

Conversation

AutorestCI
Copy link
Contributor

Add debugsend and NH patch operation
@AutorestCI
Copy link
Contributor Author

AutorestCI commented Jun 12, 2018

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-notificationhubs

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

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_3215#egg=azure-mgmt-notificationhubs&subdirectory=azure-mgmt-notificationhubs"

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

  • git checkout restapi_auto_3215
  • pip install -e ./azure-mgmt-notificationhubs

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_3215
  • pip wheel --no-deps ./azure-mgmt-notificationhubs

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

codecov-io commented Jun 12, 2018

Codecov Report

Merging #2724 into restapi_auto_notificationhubs/resource-manager will decrease coverage by 0.01%.
The diff coverage is 30.61%.

Impacted file tree graph

@@                                Coverage Diff                                 @@
##           restapi_auto_notificationhubs/resource-manager    #2724      +/-   ##
==================================================================================
- Coverage                                           55.89%   55.87%   -0.02%     
==================================================================================
  Files                                                7213     7217       +4     
  Lines                                              151649   151745      +96     
==================================================================================
+ Hits                                                84767    84795      +28     
- Misses                                              66882    66950      +68
Impacted Files Coverage Δ
...cationhubs/models/check_availability_parameters.py 100% <ø> (ø) ⬆️
...icationhubs/azure/mgmt/notificationhubs/version.py 100% <100%> (ø) ⬆️
...onhubs/models/check_availability_parameters_py3.py 100% <100%> (ø) ⬆️
...ubs/azure/mgmt/notificationhubs/models/__init__.py 100% <100%> (ø) ⬆️
...onhubs/models/notification_hub_patch_parameters.py 33.33% <33.33%> (ø)
...bs/models/notification_hub_patch_parameters_py3.py 33.33% <33.33%> (ø)
...notificationhubs/models/debug_send_response_py3.py 55.55% <55.55%> (ø)
...gmt/notificationhubs/models/debug_send_response.py 55.55% <55.55%> (ø)
...ionhubs/operations/notification_hubs_operations.py 13.79% <9.09%> (-0.85%) ⬇️
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 412e27e...019ead0. Read the comment docs.

Fix build error due to missing parameterId
Fix build errors due to mismatch parameters
Fix error with missing parameters value for DebugSend
Change DebugSendResult from string to object
@AutorestCI AutorestCI merged commit cac919e into restapi_auto_notificationhubs/resource-manager Jun 13, 2018
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #2737

@lmazuel lmazuel deleted the restapi_auto_3215 branch June 14, 2018 18:12
lmazuel pushed a commit that referenced this pull request Apr 30, 2019
* [AutoPR notificationhubs/resource-manager] Add missing debugsend and patch operation for NotificationHub resource (#2724)

* Generated from 01edec5fd7a2a02c2d9855146656bfef4c7f423f

Add debugsend and NH patch operation

* Generated from 0238c4239dec40fef6679fd36f8498b82c6157c8

Fix build error due to missing parameterId

* Generated from 2f2ca999f5033c50ea6ad7ce9a493b9d1bd5c993

Fix more errors

* Generated from 3b57f480cb371a58ae1c0feb24e6a0270c671acd

Fix build errors due to mismatch parameters

* Generated from edb81a4b9052ef17263836b0b079b7204c6a33ce

Fix error with missing parameters value for DebugSend

* Generated from 3e947825fd1f50e39862de452aa599a4e4bc74b1

Change DebugSendResult from string to object

* Generated from 65b541d42b26f877d8ec658d93e103417c2db181

Fix DebugSend build errors

* [AutoPR notificationhubs/resource-manager] typo: notificationhubs/resource-manager/Microsoft.NotificationHubs (#3961)

* Generated from c4dc554e6b35f1b4bbaac6b73f478344470ec1bf

typo: notificationhubs/resource-manager/Microsoft.NotificationHubs

- Aauthorization -> Authorization
- Availibility -> Availability
- availabile -> available
- Pproperties -> Properties
- APNS capitalization
- MPNS capitalization
- Regenrate -> Regenerate
- notificationhub -> NotificationHub
- reponse -> response

* Packaging update of azure-mgmt-notificationhubs

* Packaging update of azure-mgmt-notificationhubs

* 2.1.0

* Packaging update of azure-mgmt-notificationhubs
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.

2 participants