Data ManagementIcon indicating the project type

Data Management

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:

                  Summary Panel