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

Add isort and other pre-commit hooks to the TSSW projects

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • None
    • None

    Description

      isort is a python package to automatically sort python imports. This ticket is for adding isort and other pre-commit hooks to the TSSW projects. In order to do that, I will create a Confluence page describing what is needed to get this working.

      Attachments

        Activity

          wvreeven Wouter van Reeven added a comment - - edited https://confluence.lsstcorp.org/display/LTS/TSSW+pre-commit+hooks

          Please review the Confluence page and provide me with feedback to improve it.

          wvreeven Wouter van Reeven added a comment - Please review the Confluence page and provide me with feedback to improve it.

          In the isort section there's this statement: touches the files in the python and test directories. However, I don't see anything in the pyproject.toml settings that indicates this or is it just an automatic assumption?

          mareuter Michael Reuter added a comment - In the isort section there's this statement: touches the files in the python and test directories. However, I don't see anything in the pyproject.toml settings that indicates this or is it just an automatic assumption?
          wvreeven Wouter van Reeven added a comment - - edited

          Thanks for catching that. It actually touches all Python files in the project, including the doc and root directories. I have modifed the text accordingly.

          wvreeven Wouter van Reeven added a comment - - edited Thanks for catching that. It actually touches all Python files in the project, including the doc and root directories. I have modifed the text accordingly.

          Page reviewed and looks good to release.

          mareuter Michael Reuter added a comment - Page reviewed and looks good to release.

          People

            wvreeven Wouter van Reeven
            wvreeven Wouter van Reeven
            Michael Reuter, Tiago Ribeiro
            Michael Reuter, Tiago Ribeiro, Wouter van Reeven
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.