mapbox / mapbox/node-cpp-skel

Feature request: Make it possible to check c++ version in same place

Offen
#140 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

We currently have to put the c++ version in two place, (1) common.gypi and (2) binding.gyp:

Let's add a shared variable for c++ version that we can use in these two places.

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

Beginne mit den C++-Versionseinstellungen in common.gypi Zeile 5 und binding.gyp Zeile 110. Untersuche, wie diese beiden Dateien derzeit die Version angeben, und identifiziere anschließend eine gemeinsame Variable, die an beiden Stellen verwendet werden kann. Die Aufgabe ist erledigt, wenn die Version einmal definiert ist und beide Build-Konfigurationsstellen sie konsistent verwenden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp, node.js
Bereich
build-system
Issue-Typ
Feature
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.