corrige workflows do GitHub Actions #170
Draft
Conversation
This file contains hidden or 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
Member
vitfera
commented
Jan 8, 2026
vitfera
commented
- Adiciona espera de 10s após iniciar serviços para garantir que estejam prontos
- Corrige ordem de permissões (executadas antes de instalar dependências)
- Adiciona flag -T em todos os comandos docker compose exec para execução não-interativa
- Corrige comando MongoDB de 'generate:proxie' para 'generate:proxies'
- Simplifica workflow de style removendo instalação global redundante do phpcs
- Usa phpcs instalado via shivammathur/setup-php ao invés de caminho global
- Adiciona espera de 10s após iniciar serviços para garantir que estejam prontos - Corrige ordem de permissões (executadas antes de instalar dependências) - Adiciona flag -T em todos os comandos docker compose exec para execução não-interativa - Corrige comando MongoDB de 'generate:proxie' para 'generate:proxies' - Simplifica workflow de style removendo instalação global redundante do phpcs - Usa phpcs instalado via shivammathur/setup-php ao invés de caminho global
- PHP-CS-Fixer não suporta PHP 8.4 ainda - Adiciona setUnsupportedPhpVersionAllowed(true) no config do PHP-CS-Fixer - Muda workflow para usar PHP 8.3 em ambos os checks de style - Usa vendor/bin/php-cs-fixer ao invés de action externa - Corrige automaticamente erros de whitespace e newlines com phpcbf
- Adiciona isset() antes de verificar APP_DEBUG no bootstrap.php - Adiciona APP_DEBUG=1 no .env.example para ambiente de desenvolvimento - Evita erro 'Undefined array key APP_DEBUG' nos testes do GitHub Actions
- Permite executar com PHP 8.3 mesmo que composer.json exija PHP 8.4 - Necessário porque PHP-CS-Fixer não suporta PHP 8.4 ainda
- Downgrade PHPUnit de ^12 para ^11 para compatibilidade com ParaTest - Downgrade phpunit/php-code-coverage de ^12 para ^11 - Ajusta ParaTest para versão ^7.4 compatível com PHPUnit 11 - Corrige 14 arquivos com erros de estilo do PHP-CS-Fixer (aspas) - Resolve erro 'wasSuccessfulIgnoringPhpunitWarnings' não encontrado - Atualiza composer.lock com versões compatíveis
- Downgrade PHPUnit para ^10.5 (de ^11) - Downgrade phpunit/php-code-coverage para ^10.1 (de ^11) - Ajusta ParaTest para ~7.4.8 compatível com PHPUnit 10 - Adiciona MONGODB_URL e MONGODB_DB ao .env.test - Resolve erro 'wasSuccessfulIgnoringPhpunitWarnings' não encontrado
- Adiciona cache:clear --env=test antes de executar testes - Garante que o container do teste use as variáveis do .env.test - Resolve erro 'Environment variable not found: MONGODB_URL' nos testes
- Remove var/cache/test inteiro ao invés de usar cache:clear - Garante rebuild completo do container com .env.test - Resolve persistência de MONGODB_URL não encontrada
- Adiciona MONGODB_URL e MONGODB_DB ao .env após criação - Garante que variáveis estejam disponíveis em todos os testes - Resolve erro persistente de MONGODB_URL não encontrada
- Adiciona MONGODB_URL e MONGODB_DB em todos os jobs de teste - Garante que MongoDB esteja configurado antes de iniciar os containers
- Remove cache de teste após composer install - Garante que container seja reconstruído com variáveis MongoDB corretas - Ordem: install deps → remove cache → generate keys → migrations
- Define MONGODB_URL e MONGODB_DB via export no bash - Garante que variáveis estejam disponíveis durante execução dos testes - Bypass do sistema de .env do Symfony
Exporta DATABASE_URL junto com MONGODB_URL e MONGODB_DB no shell Resolve erro 'Environment variable not found: DATABASE_URL' Ambos workflows (tests.yml e coverage.yml) atualizados
Inclui: JWT_SECRET_KEY, JWT_PUBLIC_KEY, JWT_PASSPHRASE Inclui: STORAGE_BASE_URL, STORAGE_IMAGES_DIR, STORAGE_IMAGES_BASE_URL Resolve 585 erros de env vars não encontradas
|
salve @vitfera esse PR conflita com o outro aberto! estamos focando no suporte da aplicacao apenas para o k8s. entendo que esse PR resolve o ambiente local utilizando docker compose. eu nao tenho condicoes de testar o PR agora mas se o @dsilvadori conseguir testar, podemos fazer o merge. |
Member
Author
|
@lpirola , é apenas para testes, falta coisas pra corrigir, podemos ignorar ou futuramente ver o que fazer |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.