This repository has been archived by the owner on Feb 19, 2025. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 6
Working prototype - baseline for writing exercises for attendees #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…` dependencies Also included `doctrine/migrations` (we will use it to start the dev environment), as well as `doctrine/coding-standard` (CS rules will help us finding undocumented types) and `qossmic/deptrac` (shim version - useful to enforce dependencies)
…persistence concepts around it
…e powerful automation logic
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
Ocramius
commented
Jul 31, 2022
src/EventSourcing/Infrastructure/Projection/DbTableProjectionDefinition.php
Outdated
Show resolved
Hide resolved
Ocramius
commented
Jul 31, 2022
Can't wait for the finished product. Would like to compare it to patchlevel/event-sourcing (I am using currently). Which is a very simplistic way of using event-sourcing. (Also has some symfony integration) |
Includes: * tests * example domain and docs for it * full static analysis pass * good coverage Required adding some more dependencies: * `symfony/console` (used in `doctrine/migrations` underlying components) * `php-standard-library/psalm-plugin` for better type inference As this stands, this still needs to go through CS checks, but that will happen in a separate commit, to avoid breaking the current achieved stability.
romm
reviewed
Aug 16, 2022
…at with Laminas CI
The idea is that the same `Dockerfile` used for CI can be used for running the workshop, so we roll with that. That means a less nice CI pipeline, but good enough for our purposes. Note how CI does **NOT** push the docker image: since this is a public repository, we cannot really perform docker registry pushes consistently, since the pipeline would always fail due to missing secrets. Also, we had to revert the SQLite JSON access syntax: not yet available for PHP + Ubuntu 22.04.
…s and exercises Also changed the license to `proprietary` for now: this is not really charity work, and I plan to use it for my own purposes, for now.
Also squished some pesky mutants
An `AggregateId` can be anything now
…utility in this system
romm
reviewed
Aug 18, 2022
**Note:** in this workshop, we store the events in a relational database, but you can pick any storage | ||
technology that guarantees durable, atomic persistence of events. | ||
|
||
**Note:**: This is like blockchain, minus the bullshit. |
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.
😆
romm
reviewed
Aug 18, 2022
Co-authored-by: Romain Canon <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TODO:
configure renovate (post-merge)Makefile
for easy usageprovide bootstrappermoved to Add entrypoints for workshop attendees #2