greenwave

#78 Remove the init methods on our YAMLObject classes.
Closed: Fixed Opened by ralph.

Closed: Fixed
Reopen Issue

While experimenting, I think I discovered that the __init__ methods of the YAMLObject classes are not called at all. They are somehow side-stepped by the yaml library.

We should verify that this is true, and if it is, then delete those methods. It is only confusing to have them there if they are ignored.

Right, internally, the yaml.YAMLObject will call __new__ instead of __init__ and set the fields by using instance.__dict__. The example in the documentation[1] is misleading and it should be safe to remove them.

[1] http://pyyaml.org/wiki/PyYAMLDocumentation#Constructorsrepresentersresolvers
Edited by mjia

Fixed by #81.

Metadata Update from @ralph:
- Issue status updated to: Closed (was: Open)

Metadata Update from @dcallagh:
- Issue assigned to ralph
- Issue close_status updated to: Fixed
- Issue set to the milestone: 0.2
Metadata
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.