mapbox / mapbox/node-cpp-skel

Integrate a tool for catching missing includes

Offen
#142 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C++
Sterne
74
Forks
9
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Let's integrate a tool such as:

to catch missing includes that might not appear until Travis tries to build your code for a different OS, see https://travis-ci.org/mapbox/node-cpp-skel/builds/394326432 for example and notice how this builds for macOS but not Linux (this was due to missing #include <memory>).

Referencing @springmeyer's comment and tool suggestions here: https://github.com/mapbox/node-cpp-skel/pull/129#issuecomment-398533472

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Vergleiche include-what-you-use, include-fixer und cpplint und untersuche anschließend den in diesem Issue referenzierten Travis-Build, um festzustellen, wo eine Prüfung ausgeführt werden könnte. Erledigt ist die Aufgabe, wenn das ausgewählte Tool integriert ist und fehlende C++-Includes wie in den relevanten Builds erkennt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
build-system, ci-cd, tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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