Merge pull request #11 from eritikass/improved-username-regex

improve username validation regex to match long usernames with "-" them

Author: eritikass

slack.go

@@ -53,7 +53,7 @@ func Slack(markdown string, options ...SlackConvertOptions) string {
 	linkRegex := ".*"
 
 	// usernames @xxx
-	re = regexp.MustCompile(startOfPattern + "@(?P<USERNAME>[a-zA-Z0-9]{3,20})" + endOfPattern)
+	re = regexp.MustCompile(startOfPattern + "@(?P<USERNAME>[a-z-A-Z0-9]{3,20})" + endOfPattern)
 	markdown = re.ReplaceAllString(markdown, fmt.Sprintf("${THE_START_OF_MATCH}<%s/${USERNAME}|@${USERNAME}>${THE_END_OF_MATCH}", gitHubURL))
 
 	// This makes sure all newlines are in the correct format, since this will otherwise cause issues with further replacements

slack_test.go

@@ -118,7 +118,9 @@ func TestSlackGithubUsername(t *testing.T) {
 	assert := assert.New(t)
 
 	assert.Equal("<https://github.com/eritikass|@eritikass>", Slack("@eritikass"))
+	assert.Equal("<https://github.com/roman-shandurenko|@roman-shandurenko>", Slack("@roman-shandurenko"))
 	assert.Equal("<https://github.com/someone2Awesome8|@someone2Awesome8>", Slack("@someone2Awesome8"))
+	assert.Equal("<https://github.com/soXeo-ne2Awes-ome8|@soXeo-ne2Awes-ome8>", Slack("@soXeo-ne2Awes-ome8"))
 	assert.Equal("example@example.com", Slack("example@example.com"))
 	assert.Equal("foo <https://github.com/eritikass|@eritikass> booo!", Slack("foo @eritikass booo!"))
 }