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

Rename ap_verify.pipeline_driver

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: ap_verify
    • Labels:
      None

      Description

      Andy Salnikov pointed out in a review that ap_verify inconsistently uses camel-case and snake-case in its package names. It was intended to be camel-case everywhere. If we let the ap_verify module be a special exception (it's the entry point for the ap_verify programs, so it makes sense for it to have the same name), then pipeline_driver should be renamed to pipelineDriver.

      Since the pipeline_driver module is an implementation detail of ap_verify, no RFC is needed. However, the module is referenced by internal code, and not just in import statements.

        Attachments

          Activity

          There are no comments yet on this issue.

            People

            Assignee:
            krzys Krzysztof Findeisen
            Reporter:
            krzys Krzysztof Findeisen
            Watchers:
            Ian Sullivan, Krzysztof Findeisen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:

                Jenkins

                No builds found.