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

remove array-align #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lsmith77
Copy link

@lsmith77 lsmith77 commented Aug 27, 2019

this PR optionally uses https://github.com/themsaid/laravel-langman for dumping the translations to ensure compatibility of the formats

also handle case when a JSON file is invalid

src/PrettyVarExport.php Outdated Show resolved Hide resolved
@lsmith77 lsmith77 force-pushed the remove-array-align branch 4 times, most recently from c62292f to 1794f86 Compare September 3, 2019 07:11
@lsmith77 lsmith77 force-pushed the remove-array-align branch 2 times, most recently from 8305d37 to e41e219 Compare October 10, 2019 13:56
@lsmith77
Copy link
Author

@MichaelHoste I have cleaned up the PR a bit since you have done several improvements upstream already.

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