rapidsai/cudf

[FEA] Public header and detail header files should have the same names

Offen

#12.112 geöffnet am 09.11.2022

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (735 Forks)batch import
breakingfeature requestgood first issueimprovementlibcudf

Repository-Metriken

Stars
 (6.000 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 17T 21h) (230 gemergte PRs in 30 T)

Beschreibung

As the title said, the public and detail headers should have the same names. Currently there are some inconsistencies between them. For example:

  • Public header: cudf/copying.hpp,
  • Detail header: cudf/detail/copy.hpp.

We should change the file names to make them consistent.

Contributor Guide

Research-Richtung
Identifizieren Sie alle Paare von öffentlichen und Detail Header Dateien, bei denen die Namen unterschiedlich sind (z. B. cudf/copying.hpp vs. cudf/detail/copy.hpp). Entscheiden Sie für jedes Paar den korrekten Namen (wahrscheinlich der Name der öffentlichen Header Datei) und benennen Sie die Detail Header Datei entsprechend um. Aktualisieren Sie alle Include Anweisungen, die auf den alten Detail Header Namen verweisen. Überprüfen Sie Abhängigkeiten in Build Skripten oder Dokumentation.
Tech Stack
cpp
Domain
backenddata
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
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
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90