Github user ran-z commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/168#discussion_r124778556
  
    --- Diff: aria/modeling/relationship.py ---
    @@ -150,18 +156,23 @@ def one_to_many(model_class,
     
         *This utility method should only be used during class creation.*
     
    -    :param model_class: The class in which this relationship will be 
declared
    +    :param model_class: class in which this relationship will be declared
         :type model_class: type
    -    :param child_table: Child table name
    +    :param other_table: other table name
         :type other_table: basestring
    -    :param other_fk: Foreign key name at the child table (no need specify 
if there's no ambiguity)
    +    :param other_fk: foreign key name at the other table (no need specify 
if there's no ambiguity)
         :type other_fk: basestring
    -    :param dict_key: If set the value will be a dict with this key as the 
dict key; otherwise will
    -                     be a list
    +    :param dict_key: if set the value will be a dict with this key as the 
dict key; otherwise will
    +     be a list
         :type dict_key: basestring
    -    :param back_populates: Override name of matching many-to-one property 
at child table; set to
    -                           false to disable
    -    :type back_populates: basestring|bool
    +    :param back_populates: override name of matching many-to-one property 
at other table; set to
    +     ``false`` to disable
    --- End diff --
    
    :+1: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to