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

Remove rule on return value policy from pybind11 style guide

    XMLWordPrintable

    Details

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

      Description

      The consensus on RFC-597 was that the return value policy rule in the pybind11 style guide is unneccessary, so remove it. Developers will be expected to directly apply the pybind11 documentation concerning return value policies.

        Attachments

          Issue Links

          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-597 Update return value policy text in Pybind11 Style Guide

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Make sure to mention RVPs in the tutorial, so that new pybind11 developers are aware that they need to make a choice.

            Show
            krzys Krzysztof Findeisen added a comment - Make sure to mention RVPs in the tutorial, so that new pybind11 developers are aware that they need to make a choice.
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Hi Kian-Tat Lim, could you review these changes (including the how-to addition you requested on RFC-597)? Thanks!

            Show
            krzys Krzysztof Findeisen added a comment - Hi Kian-Tat Lim , could you review these changes (including the how-to addition you requested on RFC-597 )? Thanks!
            Hide
            Permalink
            ktl Kian-Tat Lim added a comment -

            I approve this change. Looks great, and thanks for the example.

            Show
            ktl Kian-Tat Lim added a comment - I approve this change. Looks great, and thanks for the example.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Kian-Tat Lim
              Watchers:
              Kian-Tat Lim, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.