-
Notifications
You must be signed in to change notification settings - Fork 103
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 CLI support to prepend/append #63
Comments
I suppose the reason why CLI was chosen to be excluded is because xhprof interface relies on SERVER_NAME and similar environment variables. This would require either a separate interface for the CLI script execution index or default values to something like "CLI-ENV" and use file path instead of server name. |
yes, i think we can find good analogies for those env variables when |
I think CLI support by default would be a great addition. I might need to get it working for a project, if I do I will open a PR. |
I would definitly accept a PR against https://github.com/staabm/xhprof.io for CLI support. |
👍 |
The prepend and append includes specifically do not support CLI requests. Is that a design choice or merely "noone has written it"? I'm interested in contributing this if there is interest. Any guidelines/tips would be appreciated.
The text was updated successfully, but these errors were encountered: