Clients/neveling
2
0
Fork 0
Code Issues 1 Pull requests Projects 1 Releases Packages Wiki Activity Actions

Test harness + fix getenv config bug #24

Merged
mrouissi merged 2 commits from fix/getenv-and-tests into master 2026-05-31 17:44:16 +02:00
Conversation 0 Commits 2 Files changed 17 +1932 -156
mrouissi commented 2026-05-31 17:44:14 +02:00
Owner
Copy link

Adds PHPUnit (run in CI). Fixes the critical Symfony-6 migration bug where getenv() returns false for app config (tax=0, prices=0, localhost links) by reading $_ENV. Guarded by NoGetenvTest.

Adds PHPUnit (run in CI). Fixes the critical Symfony-6 migration bug where getenv() returns false for app config (tax=0, prices=0, localhost links) by reading $_ENV. Guarded by NoGetenvTest.
fix: Read config via $_ENV instead of getenv (Symfony 6 Dotenv no longer fills getenv)
All checks were successful
ci / validate (push) Successful in 14s
Details
ci / validate (pull_request) Successful in 12s
Details
c26cb86c8d
mrouissi merged commit 0997d39437 into master 2026-05-31 17:44:16 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
  
approved-project

   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
nice2have

   
question
More information is needed

  
wontfix
This won't be fixed

No labels
approved-project
bug
duplicate
enhancement
help wanted
invalid
nice2have
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Clients/neveling!24
No description provided.