Only add column offsets when they are non-zero for GitHub.

Author: chdsbd

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## 3.3.0 - 2025-11-05
+
+### Changed
+
+- Only add column offsets when they are non-zero for GitHub.
+
 ## 3.2.2 - 2025-07-02
 
 ### Fixed

package.json

@@ -2,7 +2,7 @@
   "name": "githubinator",
   "displayName": "Githubinator",
   "description": "Quickly open files on Github and other providers. View blame information, copy permalinks and more. See the \"commands\" section of the README for more details.",
-  "version": "3.2.2",
+  "version": "3.3.0",
   "publisher": "chdsbd",
   "license": "SEE LICENSE IN LICENSE",
   "icon": "images/logo256.png",

src/providers.ts

@@ -111,6 +111,18 @@ export function pathJoin(...args: string[]): string {
 export class Github extends BaseProvider {
   DEFAULT_HOSTNAMES = ["github.com"]
   PROVIDER_NAME = "github"
+
+  buildLines({ start, end }: ISelection): string {
+    let line = `L${start.line + 1}`
+    if (start.character !== 0) {
+      line += `C${start.character + 1}`
+    }
+    line += `-L${end.line + 1}`
+    if (end.character !== 0) {
+      line += `C${end.character + 1}`
+    }
+    return line
+  }
   async getUrls({
     selection,
     head,
@@ -121,11 +133,7 @@ export class Github extends BaseProvider {
       return null
     }
     const rootUrl = `https://${repoInfo.hostname}/`
-    const { start, end } = selection
-    // Github uses 1-based indexing
-    const lines = `L${start.line + 1}C${start.character + 1}-L${
-      end.line + 1
-    }C${end.character + 1}`
+    const lines = this.buildLines(selection)
     const repoUrl = new url.URL(
       path.join(repoInfo.org, repoInfo.repo),
       rootUrl,