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

Fixed bug described in issue #351 #398

Merged
merged 1 commit into from
Sep 4, 2020
Merged

Fixed bug described in issue #351 #398

merged 1 commit into from
Sep 4, 2020

Conversation

vhatuncev
Copy link
Contributor

@vhatuncev vhatuncev commented Apr 11, 2020

#351
Changed condition to mark position as a start of new entry. Before only # character was entries delimiter.
@turquoiseowl do you have any PO files examples which I can test locally after these changes?
Also, what do you think about code refactoring for POTranslationRepository.ParseTranslationFile method? I see benefits from doing so as ability to unit test file parsing operations and create separate class for it like POFileParser or something like that. Current ParseTranslationFile method is big and can be reduced.

Changed condition to mark position as a start of new entry. Before that # character was entries delimiter
@turquoiseowl turquoiseowl merged commit 93d0e44 into turquoiseowl:master Sep 4, 2020
@turquoiseowl
Copy link
Owner

Thanks for this. My apologies for the delay - I wasn't getting github emails.
I suggest we tests via a prerelease release.

@vhatuncev
Copy link
Contributor Author

@turquoiseowl ok, what of the process for prerelease and how I can help?

@vhatuncev vhatuncev deleted the issue_351 branch September 8, 2020 11:39
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