Merge pull request #56 from sirwart/improve-slugify-compatibility

Remove leading and trailing hyphens from slugs to match jekyll behavior

utils/strings.go

@@ -43,10 +43,15 @@ func SafeReplaceAllStringFunc(re *regexp.Regexp, src string, repl func(m string)
 }
 
 var nonAlphanumericSequenceMatcher = regexp.MustCompile(`[^[:alnum:]]+`)
+var leadingOrTrailingHyphenMatcher = regexp.MustCompile(`(^-|-$)`)
 
 // Slugify replaces each sequence of non-alphanumerics by a single hyphen
 func Slugify(s string) string {
-	return strings.ToLower(nonAlphanumericSequenceMatcher.ReplaceAllString(s, "-"))
+	slug := strings.ToLower(nonAlphanumericSequenceMatcher.ReplaceAllString(s, "-"))
+
+	// remove leading and trailing hyphen
+	slug = leadingOrTrailingHyphenMatcher.ReplaceAllString(slug, "")
+	return slug
 }
 
 // StringArrayToMap creates a map for use as a set.

utils/strings_test.go

@@ -35,6 +35,7 @@ func TestSlugify(t *testing.T) {
 	require.Equal(t, "ab-c", Slugify("ab-c"))
 	require.Equal(t, "ab-c", Slugify("ab()[]c"))
 	require.Equal(t, "ab123-cde-f-g", Slugify("ab123(cde)[]f.g"))
+	require.Equal(t, "abc", Slugify("abc?"))
 }
 
 func TestStringArrayToMap(t *testing.T) {