Skip to content

Conversation

@battis
Copy link
Contributor

@battis battis commented Aug 5, 2025

No description provided.

Setting withPartitioned() will also set withSecure()
@battis battis requested a review from gsteel August 13, 2025 00:11
Co-authored-by: George Steel <george@netglue.uk>
Copy link
Collaborator

@gsteel gsteel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please fix CS issues? Otherwise LGTM. Once CI is green, I'll release when I get the chance

@battis
Copy link
Contributor Author

battis commented Aug 13, 2025

Sorry to be such a drag on itty-bitty fixes! Thanks for bearing with me.

@gsteel
Copy link
Collaborator

gsteel commented Aug 13, 2025

Sorry to be such a drag on itty-bitty fixes! Thanks for bearing with me.

No worries 👍

Try running ./vendor/bin/phpcs and ./vendor/bin/phpcbf locally

@battis
Copy link
Contributor Author

battis commented Aug 13, 2025 via email

Copy link
Collaborator

@gsteel gsteel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @battis 👍 I'll merge and release ASAP

@gsteel gsteel self-assigned this Sep 3, 2025
@gsteel gsteel merged commit f9c6387 into dflydev:main Sep 3, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants