apache / apache/datafusion-sqlparser-rs

Create an MSRV policy in this crate

Offen
#1,744 3 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
3.5k
Forks
772
Ø Merge
4 T. 9 Std.
Gemergte PRs (30 T.)
17

Beschreibung

TLDR woudl be:
1. Define a MSRV policy (maybe copy the existing DataFusion one)
2. Implement some sort of CI check (again can copy / reuse the datafusion one if we want)
3.

🤔 interestingly we don't seem to have an MSRV policy in this crate (at least not that I could find in https://github.com/apache/datafusion-sqlparser-rs/blob/main/README.md)

We do have such a thing in DataFusion here: https://github.com/apache/datafusion?tab=readme-ov-file#rust-version-compatibility-policy

Maybe it is worth considering adding some documentation / policy in the sqlparser crate too 🤔

_Originally posted by @alamb in https://github.com/apache/datafusion-sqlparser-rs/issues/1736#issuecomment-2676808035_

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die README dieses Crates mit DataFusions Rust-Version-Kompatibilitätsrichtlinie zu vergleichen, und prüfe anschließend den im Issue referenzierten bestehenden DataFusion-CI-Check. Lege die MSRV-Richtlinie des Crates fest, dokumentiere sie in der README und füge einen CI-Check hinzu, der das gewählte Kompatibilitätsniveau erzwingt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
ci-cd, documentation
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

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