Merge pull request #353 from carpentries/fix-source-url

update source url processing
carpentries · Oct 24, 2022 · cef110b · cef110b
2 parents 0abf2de + 89e1c80
commit cef110b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: sandpaper
 Title: Create and Curate Carpentries Lessons
-Version: 0.10.4
+Version: 0.10.5
 Authors@R: c(
     person(given = "Zhian N.",
            family = "Kamvar",

diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,10 @@
+# sandpaper 0.10.5
+
+## MISC
+
+* The the `source` config parameter will have the trailing `/` trimmed off when
+  passed to the site so that the URL does not have two consectutive slashes.
+
 # sandpaper 0.10.4
 
 BUG FIX

diff --git a/R/utils-yaml.R b/R/utils-yaml.R
@@ -166,7 +166,7 @@ create_pkgdown_yaml <- function(path) {
       config  = siQuote(path_config(path)),
       title   = siQuote(usr$title),
       time    = UTC_timestamp(Sys.time()),
-      source  = siQuote(usr$source),
+      source  = siQuote(sub("/$", "", usr$source)),
       branch  = siQuote(usr$branch),
       contact = siQuote(usr$contact),
       # What carpentry are we dealing with?