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

Port pex_logging to Python 3

    XMLWordPrintable

    Details

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

      Description

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

        Attachments

          Issue Links

          blocks

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

          • Won't Fix
          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: