Skip to content

Conversation

@Szpadel
Copy link
Member

@Szpadel Szpadel commented Aug 19, 2025

macfja/phar-builder is no longer maintained, it was replaced. symfony/console updated to version supporting php 8.4 and compatable with phar-builder

macfja/phar-builder is no longer maintained, it was replaced.
symfony/console updated to version supporting php 8.4 and compatable with phar-builder
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates dependencies and replaces the deprecated macfja/phar-builder with clue/phar-composer for building PHAR archives. The changes also update Symfony Console to version 6.4 for PHP 8.4 compatibility and modify the build process to use Docker containers more effectively.

Key changes:

  • Replace macfja/phar-builder with clue/phar-composer and update build configuration
  • Update symfony/console to ^6.4 and downgrade psr/log to ^1 for compatibility
  • Refactor Docker build process to use container copying instead of volume mounting

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 2 comments.

File Description
composer.json Updates dependencies and replaces phar-builder configuration with phar-composer build scripts
build.sh Refactors Docker build process to use container creation and copying instead of volume mounting
build.dockerfile Adds git safe directory configuration to handle ownership issues in Docker

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@MariuszJozwiak MariuszJozwiak merged commit 0de9cc3 into master Aug 20, 2025
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.

3 participants