Skip to content

Conversation

@serhiidonii
Copy link

No description provided.

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

📊 Code Coverage Report

✅ Coverage: 100.00% (min 70%) — OK


Generated by Macpaw Symfony PHP Reusable Workflow

Copy link

@abilyk abilyk left a comment

Choose a reason for hiding this comment

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

чи актуально
"php": "8.2"
?

@abilyk abilyk requested a review from a team December 5, 2025 12:31
Comment on lines +11 to +14
environment:
- PHP_SERVER_PORT=${PHP_SERVER_PORT:-8000}
env_file:
- .env

Choose a reason for hiding this comment

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

а нащо тут в енвах дублювати PHP_SERVER_PORT якщо він вже є в .env.example?

Copy link
Author

Choose a reason for hiding this comment

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

@alekseytupichenkov щоб легко змінювати destination port

Choose a reason for hiding this comment

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

Ну просто як на мене то це лише може збивати то де треба правити порт

Choose a reason for hiding this comment

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

Я про те що якщо я зайду в цей файл, виправлю порт на 8123, то він все одно візьме з енву якщо він там вже встановлений

Copy link
Author

Choose a reason for hiding this comment

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

@alekseytupichenkov воно так але я з тим розрахунком робив, щоб правити лише те що не в гіті щоб легко було цим керувати

Choose a reason for hiding this comment

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

Хммм, так я про це я кажу ж наче .env не в гіті, docker-compose.yml в гіті, нащо тут

    environment:
      - PHP_SERVER_PORT=${PHP_SERVER_PORT:-8000}

chmod +x scripts/update-package.sh

# Minimal: only vendor/package (namespace will be derived)
scripts/update-package.sh acme/cool-bundle

Choose a reason for hiding this comment

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

Цей скріпт не запускається в докер кентейнері бо там нема башу

Copy link
Author

Choose a reason for hiding this comment

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

@alekseytupichenkov виправи dockerfile

fi

read -r -p "Proceed with the update? [y/N] " confirm
confirm=${confirm,,}

Choose a reason for hiding this comment

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

напевно тут помилка

Suggested change
confirm=${confirm,,}
confirm=${confirm}

@@ -0,0 +1,159 @@
#!/usr/bin/env bash

Choose a reason for hiding this comment

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

А в чому взагалі ідея цього скріпту?

@@ -0,0 +1,140 @@
<?php

Choose a reason for hiding this comment

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

Не зовсім розумію нащо тут index.php якщо це темплейт для пакєтів

@@ -0,0 +1,145 @@
### Creating a new Symfony package from this template

Choose a reason for hiding this comment

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

А це ж по суті описано в рідмі

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

📊 Code Coverage Report

✅ Coverage: 100.00% (min 70%) — OK


Generated by Macpaw Symfony PHP Reusable Workflow

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

📊 Code Coverage Report

✅ Coverage: 100.00% (min 70%) — OK


Generated by Macpaw Symfony PHP Reusable Workflow

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

📊 Code Coverage Report

✅ Coverage: 100.00% (min 70%) — OK


Generated by Macpaw Symfony PHP Reusable Workflow

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

📊 Code Coverage Report

✅ Coverage: 100.00% (min 70%) — OK


Generated by Macpaw Symfony PHP Reusable Workflow

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.

4 participants