search

php-imagine / Imagine

PHP Object Oriented image manipulation library
https://imagine.readthedocs.io
Other
4.42k stars 530 forks source link

Add PHP 8.3 to CI #855

Closed dereuromark closed 9 months ago

dereuromark commented 9 months ago

And it is sufficient to test each min/max of minors, even the min/max of majors usually suffices within PHP.

mlocati commented 9 months ago

And it is sufficient to test each min/max of minors, even the min/max of majors usually suffices within PHP.

I'd disagree: it's better to test every single Major.Minor version of PHP.

dereuromark commented 9 months ago

There is no logical reason in doing so, I can say that with confidence over thousands of repos and years of experience Only creates more waiting time and burns resources :) But do what you must.

dereuromark commented 9 months ago

Your problem here seems to be of different kind anyway. I will close this then and leave this to the maintainers.

mlocati commented 9 months ago

There is no logical reason in doing so, I can say that with confidence over thousands of repos and years of experience

I can say that 35 years of coding have taught me that it's better to never assume anything :wink:

Your problem here seems to be of different kind anyway.

What do you mean?

dereuromark commented 9 months ago

Well, I still have worked professionally for decades in that specific field :) Assuming is the thing you do right now

What do you mean?

See failing CI for 8.3

mlocati commented 9 months ago

Well, I still have worked professionally for decades in that specific field :)

Me too.

Assuming is the thing you do right now

Yep, both for work (daily) and for a ton of open source projects.

I don't get the reason why you are so malign: you should reconsider the way you approach people you don't know.

dereuromark commented 9 months ago

There is no malign. I am telling you the facts and logical reason.

I find it rather not helpful to have experts giving you advice and trying to help you keeping this (currently not so well maintained) repo maintained such an answer. So count me out - good luck moving forward.