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

Port pex_logging to Python 3

        Details

        • Type: Story
        • Status: Done
        • Priority: Major
        • Resolution: Done
        • Fix Version/s: None
        • Component/s: pex_logging
        • Labels:
          None
        • Templates:
        • Story Points:
          1
        • Epic Link:
        • Team:
          Architecture

          Description

          Work required to get pex_logging working on python 3. Will also include some package cleanups.

            Issue Links

            blocks

            Story - Created by JIRA Agile - do not edit or delete. Issue type for a user story. DM-8268 Port daf_ingest to Python 3

            • Major - Major loss of function.
            • To Do
            mentioned in

            Page Loading...

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              price Paul Price added a comment -

              Only a few small comments on the GitHub PR.

              Show
              price Paul Price added a comment - Only a few small comments on the GitHub PR.
              Hide
              Permalink
              tjenness Tim Jenness added a comment -

              Kian-Tat Lim I've implemented all of the review comments from Paul Price but during those changes I reworked all the tests to be able to work with pytest. Can you please take a look at the final two commits on the PR?

              Show
              tjenness Tim Jenness added a comment - Kian-Tat Lim I've implemented all of the review comments from Paul Price but during those changes I reworked all the tests to be able to work with pytest. Can you please take a look at the final two commits on the PR?
              Hide
              Permalink
              tjenness Tim Jenness added a comment -

              Kian-Tat Lim and Hsin-Fang Chiang looked over the testing upgrades and approved them. Merged.

              Show
              tjenness Tim Jenness added a comment - Kian-Tat Lim and Hsin-Fang Chiang looked over the testing upgrades and approved them. Merged.

                People

                • Assignee:
                  tjenness Tim Jenness
                  Reporter:
                  tjenness Tim Jenness
                  Reviewers:
                  Kian-Tat Lim
                  Watchers:
                  Kian-Tat Lim, Paul Price, Tim Jenness
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development