4
57%
master: 57%

Ran 06 Jan 2018 07:09PM UTC

Files 10

Run time 0s

Badge

Embed ▾

Committed 06 Jan 2018 07:08PM UTC coverage: 57.027% (+0.1%) from 56.911%

Job # 313.4

Build Type

Pull #479

travis-ci

Committed by

web-flow

Commit Message

Fix links in tweets when there's a prefix

If a tweet had a prefix (@names that it was replying to) then the length
of these is counted in the indices that show the locations of entities
within the tweet. But we were applying those indices to the 'display'
part of the tweet that doesn't include the prefix.

So, if the tweet was:

    @bob Please meet @bill

and the prefix was `@bob `, then the indices for linking `@bill`
are something like `17,21`. But we were applying the link around
`@bill` to the display text part of the tweet, which is:

    Please meet @bill

And so the indices no longer lined up with `@bill`.

Now they do, and the same for URLs and hashtags.

Pull Request Pull Request #479: Fix links in tweets when there's a prefix

Coverage Stats

422 of 740 relevant lines covered (57.03%)

0.57 hits per line

ryanmcgrath / twython / 313 / 4
57%
master: 57%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 313.4

ryanmcgrath / twython / 313 / 4 57% master: 57%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 313.4

ryanmcgrath / twython / 313 / 4
57%
master: 57%

README BADGES
x