Skip to content
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

static analysis #147

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

static analysis #147

wants to merge 6 commits into from

Conversation

alanbem
Copy link
Member

@alanbem alanbem commented Apr 19, 2022

No description provided.

@alanbem alanbem self-assigned this Apr 19, 2022
@alanbem alanbem changed the base branch from master to event-sourced-aggregates-and-entities April 19, 2022 22:58
@alanbem alanbem force-pushed the event-sourced-aggregates-and-entities branch from 89356c9 to 02f0034 Compare May 8, 2022 19:18
- psalm set to level 7
- phpstan set to level 2
@alanbem alanbem force-pushed the static-analysis branch 3 times, most recently from 29deeea to 43a12a1 Compare June 12, 2022 10:00
- phpstan set to level 3
- phpstan set to level 4
- phpstan set to level 5
@codecov
Copy link

codecov bot commented Jun 12, 2022

Codecov Report

Merging #147 (2330c26) into event-sourced-aggregates-and-entities (4c7c7ba) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                             Coverage Diff                              @@
##             event-sourced-aggregates-and-entities      #147      +/-   ##
============================================================================
+ Coverage                                    99.96%   100.00%   +0.03%     
- Complexity                                    1067      1074       +7     
============================================================================
  Files                                          118       118              
  Lines                                         2757      2767      +10     
============================================================================
+ Hits                                          2756      2767      +11     
+ Misses                                           1         0       -1     
Impacted Files Coverage Δ
src/Domain/Aggregate/Identification.php 100.00% <ø> (ø)
src/Domain/AggregateRoot/Identification.php 100.00% <ø> (ø)
src/Domain/Event/Listener/Identification.php 100.00% <ø> (ø)
...regateRoot/Snapshotter/Storage/PostgresStorage.php 100.00% <ø> (ø)
...ture/Domain/Event/Converter/CompositeConverter.php 100.00% <ø> (ø)
src/Infrastructure/Domain/Event/InMemoryStream.php 100.00% <ø> (+0.99%) ⬆️
src/Infrastructure/Domain/Event/Subscriber.php 100.00% <ø> (ø)
...ure/Domain/Event/Subscription/DAO/Subscription.php 100.00% <ø> (ø)
...tructure/Domain/UnitOfWork/CompositeUnitOfWork.php 100.00% <ø> (ø)
src/Domain/AggregateRoot/EventSourcing.php 100.00% <100.00%> (ø)
... and 28 more

@alanbem alanbem force-pushed the event-sourced-aggregates-and-entities branch 2 times, most recently from a61ab7f to c32e884 Compare June 28, 2022 01:12
Base automatically changed from event-sourced-aggregates-and-entities to master November 19, 2022 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant