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

Unquote array shape field names #67

Merged
merged 2 commits into from
Sep 4, 2024
Merged

Unquote array shape field names #67

merged 2 commits into from
Sep 4, 2024

Conversation

ruudk
Copy link
Contributor

@ruudk ruudk commented Sep 4, 2024

Otherwise optionals don't work.

See #66

Otherwise optionals don't work.

See php-http#66
@ruudk ruudk mentioned this pull request Sep 4, 2024
src/MultipartStreamBuilder.php Outdated Show resolved Hide resolved
src/MultipartStreamBuilder.php Outdated Show resolved Hide resolved
@dbu dbu merged commit 825153b into php-http:1.x Sep 4, 2024
13 checks passed
@dbu
Copy link
Contributor

dbu commented Sep 4, 2024

we should have phpstan as a build step here, then we would have noticed 😅

something like https://github.com/php-http/httplug/blob/87779285e14780b0a85209bfed8abd9d5fe0322e/.github/workflows/static.yml#L10 if you are motivated to try that ;-)

@ruudk ruudk deleted the patch-1 branch September 4, 2024 13:27
@prakash559
Copy link

Error coming up:
vendor/php-http/multipart-stream-builder
err: has uncommitted changes

@ruudk
Copy link
Contributor Author

ruudk commented Sep 6, 2024

@prakash559 That's on your side.

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.

3 participants