EducationalTestingService / EducationalTestingService/rsmtool

Rename argument name in `configuration_parser.check_flag_column()`

Offen
#485 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Python
Sterne
71
Forks
21
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

It's confusing that the name of the argument is `flag_column` and one of the values that it takes is also `flag_column`. This makes the docstring very confusing to write.

We should rename the argument to `flag_column_name`

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, configuration_parser.check_flag_column() zu finden und die Docstring sowie die Aufrufstellen zu lesen. Benenne das Argument in flag_column_name um, aktualisiere die Dokumentation konsistent und verifiziere, dass Aufrufer und relevante Tests nicht mehr vom alten Namen abhängen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
tooling
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.