phpbb-de / phpbb-translation

German Translation of phpBB
GNU General Public License v2.0
10 stars 10 forks source link

Updates composer.json and composer.lock for composer usage #326

Closed Crizz0 closed 7 years ago

Crizz0 commented 7 years ago

Fix #325

Crizz0 commented 7 years ago

Ich überlege, ob ich noch folgenden Felder ergänze:

"support": {
        "issues": "https://github.com/phpbb-de/phpbb-translation/issues",
        "forum": "https://www.phpbb.de/community/",
},

und:

"authors": [
        {
            "name": "phpBB Deutschland e. V.",
            "email": "webmaster@phpbb.de",
            "homepage": "https://www.phpbb.de/go/ubersetzerteam"
        }
],

ergänzen soll, was meint ihr?

Philipp-K commented 7 years ago

Greift das für alle Branches? Wenn nein, währe die Version im Title ("... für phpBB 3.1") evtl. hilfreich.

Crizz0 commented 7 years ago

@Philipp-K: Das wird für alle branches ergänzt und nach oben gemerged. Dann natürlich mit einer anderen Versionsnummer.

gn36 commented 7 years ago

Kann nicht schaden. Wie sollten dann aber mal die Übersetzer Liste korrigieren. Aktuell ist da nur Philipp drin.

Am 16. Januar 2017 22:01:41 MEZ schrieb Christian Schnegelberger notifications@github.com:

Ich überlege, ob ich noch folgenden Felder ergänze:

"support": {
      "issues": "https://github.com/phpbb-de/phpbb-translation/issues",
      "forum": "https://www.phpbb.de/community/",
},

und:

"authors": [
      {
          "name": "phpBB Deutschland e. V.",
          "email": "webmaster@phpbb.de",
          "homepage": "https://www.phpbb.de/go/ubersetzerteam"
      }
],

ergänzen soll, was meint ihr?

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/phpbb-de/phpbb-translation/pull/326#issuecomment-272961189

gn36 commented 7 years ago

Ich glaube die Empfehlung ist, die Version ganz weg zu lassen, zumindest seitens composer/packagist. Aus den Tags geht die Version ja eindeutig hervor.

Vorteil wenn wir die dev Version drin lassen wäre, dass immer ein grober Versionsstand drin steht. Halte ich aber nicht wirklich für wichtig. Wenn von phpbb die Version gefordert wird könnten wir die auch automatisch beim Release rein schreiben mit dem build Skript. Ich habe aber nicht geschaut, ob der Validator das prüft, wenn müssten wir ggf. den noch austricksen.

Am 17. Januar 2017 19:46:47 MEZ schrieb Christian Schnegelberger notifications@github.com:

Crizz0 commented on this pull request.

@@ -1,6 +1,25 @@ {

Ich dachte wir schreiben dort nur bei einem release-tag die richtige Version rein, zu der es gehört z.B. phpBB 3.1.11 und lassen in dem branch, wo ständig was gepatcht wird, einen "-dev" Marker. Spricht da was gegen?

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/phpbb-de/phpbb-translation/pull/326

Crizz0 commented 7 years ago

phpbb verlangt das afaik nicht.

gn36 commented 7 years ago

Zurück zu den Versionsinfos: Ich schlage vor, wir werfen das überall raus (auch in der Readme) und schreiben das lediglich bei den Releases automatisiert mit dem build Skript rein, so dass man zumindest bei den erstellten Paketen irgend eine Möglichkeit hat hinterher zu kontrollieren, ob die Version vielleicht veraltet ist. Ob wir das dann in die composer.json oder die Readme schreiben ist mir egal.

Crizz0 commented 7 years ago

Das können wir auch machen, dann wäre das dort auch weg. Würde ich aber jetzt nicht unbedingt in dem PR sehen.

@bantu Hast du noch Änderungswünsche?