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

Update deprecation removal release references per RFC-945

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Per RFC-945, code deprecated before v26 (rc1 is now expected to be w_2023_31) can now be marked as being scheduled for removal after v26 rather than after v27 (but note that we can't actually remove it from main until eight weeks have passed as well).

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            This review should be entirely trivial, but I'm splitting it up as a way to make sure some tickets I created to track removal of some deprecated interfaces don't go unnoticed.  There are only two kinds of commits: one that updates "removed after v27" deprecation warnings to say "removed after v26", and one that adds TODO comments with ticket numbers to deprecations for those removals.  All of the removal tickets whose timelines have been adjusted are linked here, and I've updated their release blocking/blocked-by links as well.

            Tim Jenness:

            Christopher Waters:

            Ian Sullivan:

            Show
            jbosch Jim Bosch added a comment - This review should be entirely trivial, but I'm splitting it up as a way to make sure some tickets I created to track removal of some deprecated interfaces don't go unnoticed.  There are only two kinds of commits: one that updates "removed after v27" deprecation warnings to say "removed after v26", and one that adds TODO comments with ticket numbers to deprecations for those removals.  All of the removal tickets whose timelines have been adjusted are linked here, and I've updated their release blocking/blocked-by links as well. Tim Jenness : https://github.com/lsst/daf_butler/pull/868 https://github.com/lsst/pipe_base/pull/358 https://github.com/lsst/obs_base/pull/463 https://github.com/lsst/pipe_tasks/pull/820 https://github.com/lsst/skymap/pull/65 Christopher Waters : https://github.com/lsst/afw/pull/704 https://github.com/lsst/faro/pull/153 https://github.com/lsst/cp_pipe/pull/207 Ian Sullivan : https://github.com/lsst/ap_association/pull/175 https://github.com/lsst/ip_diffim/pull/275
            Hide
            tjenness Tim Jenness added a comment -

            My PRs look fine. I've removed my name from the review list.

            Show
            tjenness Tim Jenness added a comment - My PRs look fine. I've removed my name from the review list.
            Hide
            sullivan Ian Sullivan added a comment -

            I would suggest adding one more deprecation warning in ap_association, but ip_diffim looks good. Also removing my name from the reviewers list.

            Show
            sullivan Ian Sullivan added a comment - I would suggest adding one more deprecation warning in ap_association , but ip_diffim looks good. Also removing my name from the reviewers list.
            Hide
            czw Christopher Waters added a comment -

            Had to expand the default github views to find all of the ticket reference comments, but it looks like they're all there.

            As I'm last, marking the review complete.

            Show
            czw Christopher Waters added a comment - Had to expand the default github views to find all of the ticket reference comments, but it looks like they're all there. As I'm last, marking the review complete.

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Christopher Waters
              Watchers:
              Christopher Waters, Ian Sullivan, Jim Bosch, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.