Page MenuHomePhabricator

[Analytics] [Tech Debt] Combine `-dp` and `-fp` arguments in sqlfluff.py maintenance script
Open, Needs TriagePublic

Description

WMDE Analytics Request

This task was generated using the WMDE Analytics request form. Please use the task templates linked on our project page to create tasks for the team. Thank you!

Why (Context & Decision)

What problem are you trying to solve, and what decision will this analysis inform? Briefly explain the organisational or strategic context, why this matters now, and what action will be taken based on the outcome.

WMDE Analytics has an SQL linting script that uses SQLFluff called sqlfluff.py. As of now it can be passed a directory via -dp or a file via -fp.

What (Scope & Output)

Describe the specific question(s), metrics, segments, or deliverables (e.g., dashboard, deep dive, experiment analysis), including any relevant definitions or constraints.

  • Removal of the -dp and -fp arguments and combining the functionality into a single --path (-p) argument

By When (Timing & Priority)

Provide a clear deadline, any key milestones (e.g. launch, leadership review), and note if timing is flexible or fixed.

Task is low priority tech debt


Information below this point is filled out by the analyst.

Sub Tasks

A breakdown of the steps to complete this task.

See desired outputs above

Estimation

Estimate: 30 min
Actual:

Event Timeline

AndrewTavis_WMDE renamed this task from [Analytics] [Tech Debt] Combine `dp` and `fp` arguments in sqlfluff.py maintenance script to [Analytics] [Tech Debt] Combine `-dp` and `-fp` arguments in sqlfluff.py maintenance script.Wed, Jun 3, 6:08 PM
AndrewTavis_WMDE moved this task from Incoming to Tech Debt on the WMDE Analytics board.