Closed peterjc closed 7 years ago
Due to clashing class names, the test added in https://github.com/sanger-pathogens/gff3toembl/commit/37ed4f57b1bbda9bb5fec718c2b810d51f493898 was never live. This pull request activates the test with minor updates (now wrap at 80 chars, and no trailing new line).
However, the test was intended to verify #51 had been fixed - and instead shows that line-breaks are being inserted within long words.
In my pull request #53 I suggested this:
wrapper.break_long_words=False
That alone stopped the line breaks within long words, but gave overly long lines.
I've tried the combination but this does not seem to work either...
wrapper.break_long_words=False wrapper.break_on_hyphens=True
This pull request therefore just tests the current less-than-ideal behaviour, without fixing #51.
Due to clashing class names, the test added in https://github.com/sanger-pathogens/gff3toembl/commit/37ed4f57b1bbda9bb5fec718c2b810d51f493898 was never live. This pull request activates the test with minor updates (now wrap at 80 chars, and no trailing new line).
However, the test was intended to verify #51 had been fixed - and instead shows that line-breaks are being inserted within long words.
In my pull request #53 I suggested this:
That alone stopped the line breaks within long words, but gave overly long lines.
I've tried the combination but this does not seem to work either...
This pull request therefore just tests the current less-than-ideal behaviour, without fixing #51.