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

Fix: php strip whitespace #21

Closed
wants to merge 2 commits into from
Closed

Fix: php strip whitespace #21

wants to merge 2 commits into from

Conversation

yannoff
Copy link
Owner

@yannoff yannoff commented May 2, 2024

Adresses #23

Replace built-in php_strip_whitespace which does not handle PHP attributes

@yannoff yannoff force-pushed the fix/php-strip-whitespace branch 4 times, most recently from da5dd1b to 1e30e90 Compare May 4, 2024 14:49
- Implement home-made strip method
@yannoff yannoff force-pushed the fix/php-strip-whitespace branch from ac885f4 to 2834d5a Compare May 6, 2024 08:43
@yannoff yannoff force-pushed the fix/php-strip-whitespace branch from d5d1780 to dc66e68 Compare May 6, 2024 09:53
@yannoff
Copy link
Owner Author

yannoff commented May 19, 2024

See rationale in #23 (comment)

@yannoff yannoff closed this May 19, 2024
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.

1 participant