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

Some fixes #250

Merged
merged 2 commits into from
Feb 1, 2017
Merged

Some fixes #250

merged 2 commits into from
Feb 1, 2017

Conversation

j0k3r
Copy link
Collaborator

@j0k3r j0k3r commented Feb 1, 2017

Following #249

  • I converted recent files to LF ending instead of CRLF
  • some lines were breaking the parser, I commented them (let me know what do you about them)
  • some files were UTF8 with BOM, so the first line got some problem (title wasn't title)
  • some have Author instead of author

@fivefilters fivefilters merged commit a247cc5 into fivefilters:master Feb 1, 2017
@fivefilters
Copy link
Owner

Thanks for these! Should probably look for these kind of issues in the test code. Will make a note.

@j0k3r j0k3r deleted the fixes branch February 27, 2017 06:21
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.

None yet

2 participants