voryx / Thruway

PHP Client and Router Library for Autobahn and WAMP (Web Application Messaging Protocol) for Real-Time Application Messaging
MIT License
674 stars 117 forks source link

is project alive? #355

Closed fondberg closed 1 year ago

fondberg commented 2 years ago

I'm looking for a framework like this but ut seems nothing has happened in 5 months and therefore I'm wondering g if the project is alive?

WyriHaximus commented 2 years ago

AFAIK, and @mbonneau and @davidwdan can correct me on this, the project is stable with all dependencies matching covering the latest versions. So is it alive, I won't doubt it is, but is it also stable and on the slow burner, most likely.

mbonneau commented 2 years ago

Yes. We still maintain and use this project. We just have not made many changes recently. It is generally stable and updates are made mostly to keep up with newer versions of dependencies.

fondberg commented 2 years ago

Thanks!

Wirone commented 2 years ago

I wouldn't call project stable as in terms of SemVer current 0.6 version is for initial development and can't be considered as stable. Also Composer requirement php: >=5.6 is just wrong because you can't allow the same version for PHP 5, 7 and 8. On PHP8 it fails with:

rnd-thruway-1  | PHP Fatal error:  Declaration of Thruway\Logging\ConsoleLogger::log($level, $message, array $context = []) must be compatible with Psr\Log\AbstractLogger::log($level, Stringable|string $message, array $context = []): void in /thruway/vendor/thruway/client/src/Logging/ConsoleLogger.php on line 22
# composer why psr/log
thruway/client  0.5.6  requires  psr/log (^3 || ^2 || ^1)

You just can't allow psr/log v1 and v2+ at the same time since API signatures are different (see errata of the PSR). It needs to be fixed.

ChargemapHakan commented 2 years ago

I wouldn't call project stable as in terms of SemVer current 0.6 version is for initial development and can't be considered as stable. Also Composer requirement php: >=5.6 is just wrong because you can't allow the same version for PHP 5, 7 and 8. On PHP8 it fails with:

rnd-thruway-1  | PHP Fatal error:  Declaration of Thruway\Logging\ConsoleLogger::log($level, $message, array $context = []) must be compatible with Psr\Log\AbstractLogger::log($level, Stringable|string $message, array $context = []): void in /thruway/vendor/thruway/client/src/Logging/ConsoleLogger.php on line 22
# composer why psr/log
thruway/client  0.5.6  requires  psr/log (^3 || ^2 || ^1)

You just can't allow psr/log v1 and v2+ at the same time since API signatures are different (see errata of the PSR). It needs to be fixed.

Well, interesting point, we do have the same issue here. I guess we could create a pull request to fix the PHP8 compatibility issues.

WyriHaximus commented 2 years ago

I wouldn't call project stable as in terms of SemVer current 0.6 version is for initial development and can't be considered as stable. Also Composer requirement php: >=5.6 is just wrong because you can't allow the same version for PHP 5, 7 and 8. On PHP8 it fails with:

rnd-thruway-1  | PHP Fatal error:  Declaration of Thruway\Logging\ConsoleLogger::log($level, $message, array $context = []) must be compatible with Psr\Log\AbstractLogger::log($level, Stringable|string $message, array $context = []): void in /thruway/vendor/thruway/client/src/Logging/ConsoleLogger.php on line 22
# composer why psr/log
thruway/client  0.5.6  requires  psr/log (^3 || ^2 || ^1)

You just can't allow psr/log v1 and v2+ at the same time since API signatures are different (see errata of the PSR). It needs to be fixed.

Well, interesting point, we do have the same issue here. I guess we could create a pull request to fix the PHP8 compatibility issues.

Yeah that's my bad. When I looked at that package I missed that it's not just consuming the LoggerInterface but also implements it. Will file a revert PR and, supporting v1, v2, v3 will mean the implementation needs to be pulled out into another package.

treck commented 1 year ago

1 Year no activity. Is this project still alive?

mbonneau commented 1 year ago

@treck Yes - project is alive. Over 12,000 installs in the last 30 days: https://packagist.org/packages/voryx/thruway/stats

Not sure when I will be able to look into the specific issues above.