Skip to content
This repository has been archived by the owner on Sep 18, 2021. It is now read-only.

Slash-separated hash tags #39

Merged
merged 4 commits into from
Mar 28, 2012
Merged

Conversation

fionawhim
Copy link
Contributor

Adds conformance tests for auto-linking slash-separated hashtags (e.g. "#dodge/#answer"). Also adds a fragment-specific URL test to make sure that changes don't affect URL auto-linking.

Addresses: twitter-archive/twitter-text-js#43

@keitaf
Copy link
Contributor

keitaf commented Mar 27, 2012

Can you please add the following test case (the same one you added under "urls:" section) also under "all:" section (after line 650) in autolink.yml?

- description: "Autolink url with a hashtag-looking fragment"
  text: "http://www.example.com/#answer"
  expected: "<a href=\"http://www.example.com/#answer\">http://www.example.com/#answer</a>"

This is to test Autolink.autolink() method, which calls autolinkHashtags/Urls/UsernamesAndLists() all together.

@fionawhim
Copy link
Contributor Author

Done.

keitaf pushed a commit that referenced this pull request Mar 28, 2012
@keitaf keitaf merged commit 1ff214f into twitter-archive:master Mar 28, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants