Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to manually specify config file path #511

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mobot95
Copy link

@mobot95 mobot95 commented Aug 7, 2024

What does this PR do?

Add option to manually specify config file path

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 it's an extension or dependency update, make sure adding related extensions in src/global/test-extensions.php.
  • If you changed the behavior of static-php-cli, update docs in ./docs/.
  • If you updated config/xxx.json content, run bin/spc dev:sort-config xxx.

@crazywhalecc crazywhalecc added enhancement New feature or request kind/framework Issues related to CLI app framework need test This PR has not been tested yet, cannot merge now labels Aug 7, 2024
@crazywhalecc
Copy link
Owner

Sorry for late reply, I noticed this argument is not included for MacOSBuilder, and Windows PHP build process seems don't use --with-config-file-path. It would be even better if these could be improved.

@crazywhalecc crazywhalecc removed the need test This PR has not been tested yet, cannot merge now label Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request kind/framework Issues related to CLI app framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants