Skip to content

Fix global env manager, add in-ini variable parsing#807

Merged
crazywhalecc merged 1 commit into
mainfrom
fix/env-in-apply
Jun 25, 2025
Merged

Fix global env manager, add in-ini variable parsing#807
crazywhalecc merged 1 commit into
mainfrom
fix/env-in-apply

Conversation

@crazywhalecc

Copy link
Copy Markdown
Owner

What does this PR do?

Fixes #805

Checklist before merging

If your PR involves the changes mentioned below and completed the action, please tick the corresponding option.
If a modification is not involved, please skip it directly.

  • If you modified *.php or *.json, run them locally to ensure your changes are valid:
    • PHP_CS_FIXER_IGNORE_ENV=1 composer cs-fix
    • composer analyse
    • composer test
    • bin/spc dev:sort-config
  • If it's an extension or dependency update, please ensure the following:
    • Add your test combination to src/globals/test-extensions.php.
    • If adding new or fixing bugs, add commit message containing extension test or test extensions to trigger full test suite.

@crazywhalecc crazywhalecc requested a review from henderkes June 22, 2025 12:45
@crazywhalecc crazywhalecc added the kind/framework Issues related to CLI app framework label Jun 22, 2025
Comment thread src/SPC/util/GlobalEnvManager.php
Comment thread src/SPC/util/GlobalEnvManager.php
@crazywhalecc crazywhalecc merged commit 5681722 into main Jun 25, 2025
9 checks passed
@crazywhalecc crazywhalecc deleted the fix/env-in-apply branch June 25, 2025 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/framework Issues related to CLI app framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The variable in env.ini not working

2 participants