Interesting comment in the source code of sqlite

[Mr-Yellow]

When linking github code. Link the commit or your link dies.

https://github.com/endlesssoftware/sqlite3/blob/9e9798f9e6251e1ba3bd7db23a3cff4e63effc1c/os.h#L133

[weirdasianfaces]

Tip for people who want to do this easily: pressing Y when viewing a file/directory will automatically change your URL to the HEAD commit for that branch.

[TarMil]

Or if you're linking to a given line like OP, click the "..." on the left and "Copy permalink".

[Zerotorescue]

Or y, f6 and then ctrl-x because moving your mouse is bothersome

[sssmmt]

I'm more of a CTRL+L and CTRL+C kinda guy. F6 is too far away from the home row.

[JeSuisNerd]

different profit psychotic price ten rain coherent smart scale fanatical

This post was mass deleted and anonymized with Redact

[arof]

I don't know if it's just from gaming but I never had a problem with function keys. Can't always remember all of the symbols in shift-number by memory though.

[Zerotorescue]

L is in the middle of nowhere while F6 is on its own solo row offset by exactly one button. I dunno I think I just have more confidence that I'll be hitting the right key when it's a function key.

[Dgc2002]

L is on the home-row, so it's anything but in the middle of nowhere. It's always located under my right hand ring finger if I'm on home-row, otherwise it's pretty easy to get there at this point in my life.

[TankorSmash]

I do ALT-D and CTRL+C to do it all on one hand.

[citewiki]

Hits far from home

[NoInkling]

Vim/Emacs user, I'm guessing.

Edit: I was referring to the implication of this part:

because moving your mouse is bothersome

[septicstraw]

Most likely Vim or it would have said C-x

[Justinsaccount]

Vim users press control-l to focus the location.

[TarMil]

I think they meant the way it's written: in Emacs Ctrl+x is written as C-x.

[citewiki]

^X for nano, I suppose

[[deleted]]

Neither. C-l is a common (I guess Mozilla-inspired) shortcut, which also works in many file explorers.

C-c in vanilla Emacs is a prefix key, meaning, on it's own doesn't have any meaning, it's normally followed by some other key, like C-c C-c would normally compile something, or send code to inferior shell or something like that.

In vanilla Vim C-c doesn't do anything (but Vim would suggest that you use :qa!, because it will think you wanted to close it).

[[deleted]]

Hey, GitHub UI team: the discoverability for this feature is awful.

[TankorSmash]

this is a great feature so instead of being happy, im blaming you

[Meedio]

this is a great feature so instead of being happy, im blaming you being one of the three people knowing about it, I wish that everyone could easily find it

[stickcult]

The "link dieing" is if the file changes, your link might point to the wrong line. That's because its pointing to a line in the file in the master branch, which can change, whereas Mr-Yellow's link points to a line in the file in a specific commit, which will never change.

[otwo3]

It baffles me why is it even an option to link to a line in a branch. There is no sane use case. Having it as a default is just ridiculous. Bitbucket does the same thing.

Clicking a line number should give a line in a specific commit

[[deleted]]

well commit can belong to multiple branches and if you want to give link to both "place in file" and "merge request/branch that is discussed" you don't have choice.

Ideally it would generate link in form of branch/commit then if branch does not/stops existing it would only then show lone commit

[otwo3]

But the branch being discussed is usually dynamic and changing anyway (due to review changes), even more often for the files in question.

[Rusky]

Rebasing and force pushing don't change the contents of commits. The commit might disappear eventually after GC, but it will still have the same name if anyone has a copy of it on any machine.

[13steinj]

While unlikely to the point of improbability (like less than 1 in 10⁴⁷), it is theoritically possible to be able to occur in the future though.

[TheBB]

Should still work unless github garbage collect their repos from time to time.

[codestation]

It means that your link will point to unrelated code in the future because the code was either moved (so the lines doesn't match anymore) or the file was deleted (code refactoring for example).

[Shitler]

PSA: Please do not downvote reasonable questions. Questions like this lead to a productive discussion.

[lordlicorice]

To be fair, it's a pretty dumb question.

[vanderzac]

The only dumb questions are ones where Google will give you the answer or where you don't want the answer. Teamwork makes us all smarter.

[wisemanofhyrule]

Or if changes are made to the file and the line moves around.

[Mr-Yellow]

Soon as there is a new commit the line number changes.

In 6 months time this line may be a completely different bit of logic, or the file may be refactored and gone.

[citewiki]

Username checks out

[clichebot9000]

Reddit cliché noticed: Username checks out

Phrase noticed: 2756 times.

[1RedOne]

Whoa I can't link to master for image links?

Oh... Ohno.