Skip to content

Fix JSON encoding in default config#2552

Merged
dantleech merged 1 commit intophpactor:masterfrom
MatmaRex:default-config
Feb 21, 2024
Merged

Fix JSON encoding in default config#2552
dantleech merged 1 commit intophpactor:masterfrom
MatmaRex:default-config

Conversation

@MatmaRex
Copy link
Contributor

$this->schemaPath always contains backslashes on Windows, and presumably may contain backslashes and quotes on Linux too.

@dantleech
Copy link
Collaborator

dantleech commented Feb 21, 2024

image

🤔

@dantleech
Copy link
Collaborator

happy to merge this, but there are CI issues, you can run composer integrate to run most of the checks that happen in CI (it will also fix the CS). You'll probably need to handle the case where json_encode returns NULL.

`$this->schemaPath` always contains backslashes on Windows, and
presumably may contain backslashes and quotes on Linux too.
@dantleech dantleech merged commit f2faa8e into phpactor:master Feb 21, 2024
@MatmaRex
Copy link
Contributor Author

I haven't gotten composer integrate running yet, but I'll have a look at that next.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants