search

AndiLeni / fonts

REDAXO Fonts Addon
MIT License
12 stars 4 forks source link

PHP8 Code aber kein min Version in package.yml gesetzt #13

Closed bitshiftersgmbh closed 1 year ago

bitshiftersgmbh commented 1 year ago

Bei einem Kundenprojekt knallt es, weil das Probleme macht ...

public static function getLicense(String $fontName): string|false

Das ist wohl PHP 8.x Notation. In der package.yml gibt es keine PHP-Bedingungen. Da müsste man sich in 1 Richtung entscheiden.

AndiLeni commented 1 year ago

Ja, das stimmt. Man kann es in die package.yml setzen, aber am 28 Nov 2022 ist mit der 7.4 auch offiziell Schluss.

alxndr-w commented 1 year ago

@bitshiftersgmbh die Zeit für das issue hätte man doch in einen PR investieren können

bitshiftersgmbh commented 1 year ago

@bitshiftersgmbh die Zeit für das issue hätte man doch in einen PR investieren können

Ich setz meine Zeit ein, wie ich es möchte.

alxndr-w commented 1 year ago

@bitshiftersgmbh jo, ich weiß, deine Zeit ist wertvoller als die der anderen. 😡

Ist btw. schon drin. https://github.com/AndiLeni/fonts/blob/5d3a959a777fd2167f133e379ca60d45ee98336d/package.yml#L17

close plz