Riccardo Fiorenza comments

Results 18 comments of


                                            Riccardo Fiorenza

Add CLI param to group output by rules

@AlessandroMinoccheri It's true that the others tools does not grouping by rules, but its are focused on single classes/files instead on the whole application. In my opinion PHKArkitect should be...

implement except(...) clause to improve expressivity

In my opinion will be great have a method such as `Except` instead of to pass the expression exceptions in a parameter. I don't know if it is applicable to...

implement except(...) clause to improve expressivity

Well, with the new feature added in the https://github.com/phparkitect/arkitect/pull/220 I have solved all my issues updating my rule as follow: ``` $rules[] = Rule::allClasses() ->that(new ResideInOneOfTheseNamespaces('App\Domain')) ->should(new NotHaveDependencyOutsideNamespace('App\Domain', [ 'DateTimeImmutable',...

implement except(...) clause to improve expressivity

Sure, I could also grouping the namespaces in less rows, putting more in a row, but the question is not this. The real question is: Has it sense to consider...

add optional argument to check only a specific path or file

Sure, you can change the path using `ClassSet::fromDir(...)` in the configuration but I would like to temporarily change the path to check at runtime to focus only on one file...

[BUG] When there are periods of high concurrent requests, the response processing is slow, and the time intervals are long, sometimes AxiosError: socket hang up ，code: ECONNRESET

+1 Same issue after upgrading from 1.6.0 to 1.6.8 (now 1.7.2) Maybe this one https://github.com/axios/axios/issues/6113 is related.

HTTP Keep-alive, "socket hang up" ECONNERR on a long-running second request

Translation Error: The intl string context variable "message" was not provided to the string "{otp_secret} is invalid. {message}"

Hi @alandtse, thanks for the clarification in PR https://github.com/alandtse/alexa_media_player/pull/3191#issuecomment-3594345989 regarding the auto-generated translations. I’m also affected by the same issue reported here: ``` Validation of translation placeholders for localized (it)...