tursodatabase/turso-cli

Better error handling in csv table name

Offen

#810 geöffnet am 08.03.2024

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (50 Forks)github user discovery
buggood first issue

Repository-Metriken

Stars
 (302 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4h 32m) (2 gemergte PRs in 30 T)

Beschreibung

You can create a new db from CSV:

turso db create --from-csv test12_sanitized.csv --csv-table-name some_table
  1. If someone sends an invalid character for csv-table-name param, it just prints an error as something went wrong:
Error: could not load csv into new database file: exit status 1: 4572726f723a206e65617220222d223a2073796e746178206572726f720a

We should display the proper error

  1. We update our CLI to not to accept any special characters in csv-table-name param

reported here: https://discord.com/channels/933071162680958986/1215308443217563668/1215308443217563668

Contributor Guide

Research-Richtung
Untersuchen Sie die CSV Tabellennamenverarbeitung in der CLI; lokalisieren Sie den Fehlerweiterleitungspfad im create Befehl; fügen Sie eine Eingabevalidierung hinzu, um Sonderzeichen in csv table name abzulehnen, und verbessern Sie Fehlermeldungen, um die tatsächliche Ursache anstelle von hexadezimalen Fehlercodes anzuzeigen.
Tech Stack
go
Domain
cli
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60