-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
add constructor parameters to data classes #12
Comments
Verified as implemented. |
AutorestCI
added a commit
that referenced
this issue
Mar 13, 2019
Nfsaas 1875 swagger update (#12) * NFSAAS-1875 Update to R3.5 Standard * NFSAAS-1875 update to R3.5 standard
AutorestCI
added a commit
that referenced
this issue
Mar 20, 2019
…GA) (#4556) * Generated from 46f3cb6030777135484c1bdc4b1d2146053172f9 NFSAAS-1875 Update to R3.5 Standard (#11) * Generated from 084552639aef23de670f0dc26ba5c9496eb6daff Nfsaas 1875 swagger update (#12) * NFSAAS-1875 Update to R3.5 Standard * NFSAAS-1875 update to R3.5 standard * Generated from 0af27c76c27c57415609e59ba20027b3808ba30e Nfsaas 1875 swagger update (#13) * NFSAAS-1875 Update to R3.5 Standard * NFSAAS-1875 update to R3.5 standard * NFSAAS-1875 updated from review comments
lmazuel
pushed a commit
that referenced
this issue
Mar 26, 2019
* [AutoPR netapp/resource-manager] Update to Release 3.5 standard (pre GA) (#4556) * Generated from 46f3cb6030777135484c1bdc4b1d2146053172f9 NFSAAS-1875 Update to R3.5 Standard (#11) * Generated from 084552639aef23de670f0dc26ba5c9496eb6daff Nfsaas 1875 swagger update (#12) * NFSAAS-1875 Update to R3.5 Standard * NFSAAS-1875 update to R3.5 standard * Generated from 0af27c76c27c57415609e59ba20027b3808ba30e Nfsaas 1875 swagger update (#13) * NFSAAS-1875 Update to R3.5 Standard * NFSAAS-1875 update to R3.5 standard * NFSAAS-1875 updated from review comments * Generated from c219e4e969a6443efa588eadc8ed77f611baa53e (#4638) Merge pull request #15 from leonardbf/NFSAAS-1875-correction NFSAAS-1875 small correction * updated versioning and change log * NFSAAS-2236 update python SDK tests for R3.5 (#4684) * Update HISTORY.rst
AutorestCI
added a commit
that referenced
this issue
Jun 4, 2019
Merge pull request #12 from moreOver0/ad_updates_fork change time range to time point
kristapratico
added a commit
to kristapratico/azure-sdk-for-python
that referenced
this issue
Oct 19, 2019
fixes tests, adds docstrings to wrapper classes
KieranBrantnerMagee
pushed a commit
to KieranBrantnerMagee/azure-sdk-for-python
that referenced
this issue
Apr 3, 2020
…bus/track2-migration-guide-improvement migration doc, changelog and version cleanup
beltr0n
pushed a commit
to beltr0n/azure-sdk-for-python
that referenced
this issue
Jan 22, 2021
Bertong extract identity
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Data classes for storage and service bus, such as Queue, Topic, Subscription, Rule, etc. should have additional constructor parameters with default values. This will allow creation of an instance in one statement. This should be done for the data classes that are instantiated by the user.
Old:
rule = Rule()
rule.filter_type = 'SqlFilter'
rule.filter_expression = 'messagenumber > 3'
New:
rule = Rule(filter_type='SqlFilter', filter_expression='messagenumber>3')
New (alt):
rule = Rule('SqlFilter', 'messagenumber>3')
The text was updated successfully, but these errors were encountered: