Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-9087

Add Python property guidelines to dev guide

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      Implements most of RFC-279 (all but the part that goes in the pybind11 guide).

        Attachments

          Issue Links

            Activity

            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            In which epic should this go?

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - In which epic should this go?
            Hide
            jbosch Jim Bosch added a comment -

            Ready for review. This is just a markup reformatting of the text in RFC-279, pasted into the "Idiomatic Python" section of the Python style guide.

            Rendered text can be found here:

            https://developer.lsst.io/v/DM-9087/coding/python_style_guide.html#properties-should-be-used-when-they-behave-like-regular-instance-attributes

            Show
            jbosch Jim Bosch added a comment - Ready for review. This is just a markup reformatting of the text in RFC-279 , pasted into the "Idiomatic Python" section of the Python style guide. Rendered text can be found here: https://developer.lsst.io/v/DM-9087/coding/python_style_guide.html#properties-should-be-used-when-they-behave-like-regular-instance-attributes
            Hide
            ktl Kian-Tat Lim added a comment -

            While this is indeed a transcription of the text in the RFC, I have a couple of minor requests for clarification. Otherwise, good to go.

            Show
            ktl Kian-Tat Lim added a comment - While this is indeed a transcription of the text in the RFC, I have a couple of minor requests for clarification. Otherwise, good to go.

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Kian-Tat Lim
              Watchers:
              Jim Bosch, Jonathan Sick, Kian-Tat Lim, Pim Schellart [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.