Crell / Serde

Robust Serde (serialization/deserialization) library for PHP 8.
Other
299 stars 14 forks source link

Upgrade docker setup: #2

Closed IceRidder closed 2 years ago

IceRidder commented 2 years ago

Description

Update docker to php:8.1.0 Install composer and dependencies in the docker image Remove "vendor/bin/phpunit" command from the image to allow installing composer dependencies from fresh Add composer install example command Update docker-compose commands to use "-u $(id -u ${USER}):$(id -g ${USER})" to have files generated in docker have the proper permission on the host machine Clean php80 commands Add volume (commented by default) to take advantage of local composer cache

Motivation and context

Why is this change required? What problem does it solve?

fixes#1

If it fixes an open issue, please link to the issue here (if you write fixes #num or closes #num, the issue will be automatically closed when the pull is accepted.)

How has this been tested?

Please describe in detail how you tested your changes.

Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code, etc.

Screenshots (if appropriate)

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

Crell commented 2 years ago

Thanks! I'll take care of the other tasks on this and related repos on Monday.