search

TYPO3-Caretaker / caretaker_instance

TYPO3 extension caretaker_instance
GNU General Public License v2.0
14 stars 23 forks source link

Update TYPO3 requirements #64

Closed janhelke closed 6 years ago

janhelke commented 6 years ago

The extensions works also with TYPO3 9 LTS

etobi commented 6 years ago

@janhelke Du hast das nimut/testing-framework aus require-dev entfernt. Das führt dazu, dass die Travis-Tests fehlschlagen.

Gab es einen bestimmten Grund das rauszunehmen? Soweit ich sehe dürfte das Package bereits 9.5-kompatibel sein.

IchHabRecht commented 6 years ago

Aus meiner Sicht muss auch die Travis-Konfiguration angepasst werden, um gegen die 9.5 entsprechend zu testen. Nur eine Erhöhung des requirements reicht aus meiner Sicht hier nicht.

etobi commented 6 years ago

@IchHabRecht ist das nimut/testing-framework bewusst nicht PHP 7.2 kompatibel markiert? (siehe Travis hier)

IchHabRecht commented 6 years ago

Man muss die richtige Version vom nimut/testing-framework (https://packagist.org/packages/nimut/testing-framework) nehmen, um es mit TYPO3 9.5 nutzen zu können. Aus meiner Sicht sollte als constraint "^2.0 || ^4.0" funktionieren (für TYPO3 6.2 - 9.5)

IchHabRecht commented 6 years ago

With the changes from https://github.com/TYPO3-Caretaker/caretaker_instance/pull/65 I was able to update the composer constraints to 9.5 with running tests (https://travis-ci.org/IchHabRecht/caretaker_instance/builds/455436062). I didn't tested the instance in a TYPO3 9.5 yet, so I don't know if it is really compatible or just doesn't have any test failures.

etobi commented 6 years ago

solved by #65 and #66

etobi commented 6 years ago

Danke, Jan, das Du das Thema auf den Tisch gebracht hast und wir so kurzfristig die _instance kompatibel machen konnten. Danke, Nicole, dass Du den Change direkt aufgegriffen und gelöst hast.