Fix forgejo links

Fixes #4688 (comment)

Author: lpil

CHANGELOG.md

@@ -146,3 +146,7 @@
 - Fixed a bug where the compiler would suggest a discouraged project name as an
   alternative to the reserved `gleam` name.
   ([Giacomo Cavalieri](https://github.com/giacomocavalieri))
+
+- Fixed a bug where Forgejo source URLs in the HTML documentation could be
+  incorrectly structured.
+  ([Louis Pilfold](https://github.com/lpil))

compiler-core/src/docs/source_links.rs

@@ -73,7 +73,7 @@ impl SourceLinker {
                     let cleaned_host = string_host.trim_end_matches('/');
                     Some((
                         format!("{cleaned_host}/{user}/{repo}/src/tag/{tag}/{path_in_repo}#L",),
-                        "-".into(),
+                        "-L".into(),
                     ))
                 }
                 Repository::Custom { .. } => None,
@@ -85,6 +85,7 @@ impl SourceLinker {
             url_pattern,
         }
     }
+
     pub fn url(&self, span: SrcSpan) -> String {
         match &self.url_pattern {
             Some((base, line_sep)) => {

compiler-core/src/docs/tests.rs

@@ -1190,3 +1190,47 @@ pub fn aaaaaaaaaaaaaaaaaaaaaaaaaaaa() -> aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa {
         NONE
     );
 }
+
+#[test]
+fn forgejo_single_line_definition() {
+    let mut config = PackageConfig::default();
+    let repo = Repository::Forgejo {
+        host: "https://code.example.org/".parse::<Uri>().unwrap(),
+        user: "wibble".into(),
+        repo: "wobble".into(),
+        path: None,
+        tag_prefix: None,
+    };
+
+    config.name = EcoString::from("test_project_name");
+    config.repository = Some(repo);
+
+    let modules = vec![("app.gleam", "pub type Wibble = Int")];
+    let html = compile(config, modules);
+
+    assert!(
+        html.contains("https://code.example.org/wibble/wobble/src/tag/v0.1.0/src/app.gleam#L1")
+    );
+}
+
+#[test]
+fn forgejo_multiple_line_definition() {
+    let mut config = PackageConfig::default();
+    let repo = Repository::Forgejo {
+        host: "https://code.example.org/".parse::<Uri>().unwrap(),
+        user: "wibble".into(),
+        repo: "wobble".into(),
+        path: None,
+        tag_prefix: None,
+    };
+
+    config.name = EcoString::from("test_project_name");
+    config.repository = Some(repo);
+
+    let modules = vec![("app.gleam", "pub type Wibble \n\n= Int")];
+    let html = compile(config, modules);
+
+    assert!(
+        html.contains("https://code.example.org/wibble/wobble/src/tag/v0.1.0/src/app.gleam#L1-L3")
+    );
+}