This makes assignment 9 a bit easier.

It will make moving the classes around much easier.

Or: mention that we need a root namespace in the assignment.

We can't make the container just yet, because e.g. RsvpForMeetupController still uses the dependencies stand-alone (the logic needs to be wrapped in an application service there, maybe do it before the workshop even starts?).

$meetup->events[] =

Also: use public function __construct() instead of static factory method.

It used to make sense because of older assignments that are no longer there

In ListMeetupsController

ScheduledDate is a nice example :)

• Language levels
• PHP 7.1

5: user-facing validation is generally not reusable, so let's not make it reusable ;)
10: who annotates ports anyway?

Are Cli and Web ports? Or adapters for the UI port?

Don't ask again to create those subdirectories!

First: implement getById() using fromDatabaseRecord. This means the public constructor should be changed into a public static schedule method (do this in the earlier assignment already).

Also: drop the unit tests for Meetup since they get in the way when doing all this refactoring...

In the presentation mainly.

docker-compose run --rm devtools deptrac analyze --formatter-graphviz-dump-image=/opt/var/dependency-graph.png

Should be MeetupOrganizingInterface

So we really are only refactoring.

To make sure that we don't have to introduce a Clock abstraction (yet).

Not a very relevant concept

Having resource files inside autoloaded src/ is actually an anti pattern
See also https://github.com/php-pds/skeleton
They should be under top level /resources etc.

It also makes it easier to work with the autoloaded path, as it now only contains valid autoloadable PHP files it is also faster to autoload.

For the course of the workshops it also makes it a bit more easy to work with the classes in src/.
I was bothered by that folder being there :)

Deptrac can't handle it.

And make a point about construction from \DateTimeImmutable in the infrastructure layer (another assignment)?

Otherwise we'd have to implicitly use __toString().