-
-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merging v2.13 into v3.0-dev #677
Conversation
msmakouz
commented
Apr 29, 2022
Q | A |
---|---|
Bugfix? | ❌ |
Breaks BC? | ❌ |
New feature? | ❌ |
Co-authored-by: Maxim Smakouz <m.smakouz@gmail.com>
Co-authored-by: Aleksei Gagarin <roxblnfk@ya.ru>
Update psr/log dependency version
Co-authored-by: roxblnfk <roxblnfk@ya.ru>
[ci skip] [skip ci]
…eature/broadcasting-2.0
- Fixes tests for LogBroadcast. - Gets rid of BroadcastConfig on AuthorizationMiddleware.
Adding chunkSize to the HTTP config
Improves broadcasting guard.
Adding Queueable attribute
Make dumper deprecated
…-master # Conflicts: # CHANGELOG.md # composer.json # monorepo-builder.php # src/AnnotatedRoutes/composer.json # src/Annotations/composer.json # src/Annotations/src/AnnotatedClass.php # src/Annotations/src/AnnotatedMethod.php # src/Annotations/src/AnnotatedProperty.php # src/Attributes/composer.json # src/Auth/composer.json # src/AuthHttp/composer.json # src/Boot/composer.json # src/Boot/src/AbstractKernel.php # src/Bridge/DataGrid/README.md # src/Bridge/DataGrid/composer.json # src/Bridge/DataGrid/src/Annotation/DataGrid.php # src/Bridge/DataGrid/src/Bootloader/GridBootloader.php # src/Bridge/DataGrid/src/Config/GridConfig.php # src/Bridge/DataGrid/src/GridInput.php # src/Bridge/DataGrid/src/Interceptor/GridInterceptor.php # src/Bridge/DataGrid/src/Response/GridResponse.php # src/Bridge/DataGrid/src/Response/GridResponseInterface.php # src/Bridge/Dotenv/composer.json # src/Bridge/Monolog/composer.json # src/Bridge/Stempler/composer.json # src/Cache/composer.json # src/Config/composer.json # src/Console/composer.json # src/Cookies/composer.json # src/Core/composer.json # src/Csrf/composer.json # src/DataGrid/composer.json # src/Debug/composer.json # src/Distribution/composer.json # src/Dumper/composer.json # src/Encrypter/composer.json # src/Exceptions/composer.json # src/Exceptions/src/HtmlHandler.php # src/Files/composer.json # src/Filters/composer.json # src/Framework/Bootloader/Broadcast/BroadcastBootloader.php # src/Framework/Bootloader/CommandBootloader.php # src/Framework/Bootloader/Http/HttpBootloader.php # src/Framework/Bootloader/Http/WebsocketsBootloader.php # src/Framework/Broadcast/Config/WebsocketsConfig.php # src/Framework/Broadcast/Middleware/WebsocketsMiddleware.php # src/Framework/Command/Database/ListCommand.php # src/Framework/Command/Database/TableCommand.php # src/Framework/Command/GRPC/GenerateCommand.php # src/Framework/Command/GRPC/ListCommand.php # src/Framework/Command/Migrate/AbstractCommand.php # src/Framework/Command/Migrate/InitCommand.php # src/Framework/Command/Migrate/MigrateCommand.php # src/Framework/Command/Migrate/ReplayCommand.php # src/Framework/Command/Migrate/RollbackCommand.php # src/Framework/Command/Migrate/StatusCommand.php # src/Framework/GRPC/Exception/CompileException.php # src/Framework/GRPC/GRPCDispatcher.php # src/Framework/GRPC/LocatorInterface.php # src/Framework/GRPC/ProtoCompiler.php # src/Framework/GRPC/ServiceLocator.php # src/Framework/Http/LegacyRrDispatcher.php # src/Framework/Http/RrDispatcher.php # src/Framework/Http/SapiDispatcher.php # src/Hmvc/composer.json # src/Http/composer.json # src/Http/src/Emitter/SapiEmitter.php # src/Http/src/EmitterInterface.php # src/Http/src/Exception/EmitterException.php # src/Http/src/SapiRequestFactory.php # src/Logger/composer.json # src/Mailer/composer.json # src/Models/composer.json # src/Pagination/composer.json # src/Prototype/composer.json # src/Prototype/src/Bootloader/PrototypeBootloader.php # src/Queue/composer.json # src/Reactor/composer.json # src/Router/composer.json # src/Scaffolder/composer.json # src/Security/composer.json # src/SendIt/composer.json # src/Session/composer.json # src/Snapshots/composer.json # src/Stempler/composer.json # src/Storage/composer.json # src/Streams/composer.json # src/Tokenizer/composer.json # src/Translator/composer.json # src/Validation/composer.json # src/Views/composer.json
Codecov Report
@@ Coverage Diff @@
## 3.0 #677 +/- ##
============================================
+ Coverage 82.03% 82.06% +0.02%
- Complexity 5426 5492 +66
============================================
Files 611 623 +12
Lines 13608 13755 +147
============================================
+ Hits 11164 11288 +124
- Misses 2444 2467 +23
Continue to review full report at Codecov.
|
private bool $success; | ||
private array $topics; | ||
private array $attributes; | ||
private ?ResponseInterface $response; | ||
|
||
public function __construct( | ||
bool $success, | ||
array $topics, | ||
array $attributes = [], | ||
?ResponseInterface $response = null | ||
) { | ||
$this->success = $success; | ||
$this->topics = $topics; | ||
$this->attributes = $attributes; | ||
$this->response = $response; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be replaced with promouted properties;
Also getters can be replaced with public readonly properties