-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates composer to version 1.0.0 #294
Conversation
Hat es einen besonderen Grund, wieso wir nicht auch andere PHP-Version als:
Testen? @gn36 Mit PHP 5.3.3 haben wir jetzt allerdings wieder diesen tollen Fehler, den wir im phpbb-movemessage noch durch das Entfernen von self-update entfernt haben. |
68f3ba2
to
a80fda9
Compare
Scheint die Sache zu beheben. 😄 |
Für mich wäre die Frage eigentlich fast eher, warum wir mit beiden Versionen testen. Das einzige, worauf wir bei den Sprachpaketen testen, ist ob die Dateien valide sind (phplint) und ob die Inhalte zusammenpassen (translation validator). PHPlint braucht mindestens PHP 5.5, aber damit wäre auch Array syntax erlaubt, die mit 5.3 nicht geht. Daher wohl noch mal der Test mit 5.3.3. Hier wird PHPlint wohl komplett übersprungen, aber in 5.3.3 führt die Variante mit eckigen Klammern zu einem Syntaxfehler. Weitere PHP Versionen testen dann letztlich nur das Testpaket, da die Sprachdateien nur aus Arrays bestehen. Was mich allerdings etwas irritiert ist, dass es keinerlei Meldungen von phplint gibt, weder positive noch Fehler. Ich hätte erwartet, dass es irgend eine Meldung gibt, oder eine Fehlermeldung dass phplint mindestens Version X braucht. |
Soll ich mal eine Änderung einer PHP-Datei zum testen mit in den PR packen, die einen Fehler enthält und schauen, was lint dann sagt? Am 11. April 2016 23:38:16 MESZ, schrieb Martin Beckmann notifications@github.com:
|
Habe ich auch schon überlegt. Ist vielleicht ne gute Idee. Mach das aber in nem separaten PR zum testen. |
Test ist erfolgt und geht. Ergo kann das hier rein. |
Updates the composer.phar to version 1.0.0 (first stable)