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

Add pybind11 rules to DM dev-guide

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      DRP S17-2, DRP S17-3, DRP S17-4
    • Team:
      Data Release Production

      Description

      Update DMTN-024 with conventions acquired during pybind11 development.
      These guidelines will then need to be RFC'd before being made an official part of the DM developer guide.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            Oops! Sorry about that, closed the wrong ticket.

            Show
            krzys Krzysztof Findeisen added a comment - Oops! Sorry about that, closed the wrong ticket.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Russell Owen wrap sounds slightly off to my ear, but I don't have a strong preference.

            Show
            krzys Krzysztof Findeisen added a comment - Russell Owen wrap sounds slightly off to my ear, but I don't have a strong preference.
            Hide
            ktl Kian-Tat Lim added a comment -

            What's there now looks fine to me, modulo some minor tweaks in the PR. I trust that any changes Jim will ask for are equally fine (or will fix up afterwards if not).

            Show
            ktl Kian-Tat Lim added a comment - What's there now looks fine to me, modulo some minor tweaks in the PR. I trust that any changes Jim will ask for are equally fine (or will fix up afterwards if not).
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Overlapping reviews with DM-9098, but all changes have now been implemented.
            Will be merged together with pybind11 switch.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Overlapping reviews with DM-9098 , but all changes have now been implemented. Will be merged together with pybind11 switch.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Merged

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

              People

              Assignee:
              pschella Pim Schellart [X] (Inactive)
              Reporter:
              pschella Pim Schellart [X] (Inactive)
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, John Swinbank, Jonathan Sick, Kian-Tat Lim, Krzysztof Findeisen, Pim Schellart [X] (Inactive), Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: