Skip to content

Conversation

felixfbecker
Copy link
Contributor

This is always shown in the status bar, which comes with a negative connotation for any code that is being viewed (as someone is to be blamed for it). It's more neutral to state that this is the "author" of the code, without implying they are to be "blamed" for anything (even if the underlying Git CLI command is called git blame).

@felixfbecker felixfbecker requested a review from tjkandala April 14, 2021 10:39
@felixfbecker
Copy link
Contributor Author

@tjkandala CI shows up as failed but doesn't link to any actual build. Could you investigate this (assuming extensibility owns this repo)? We might want to migrate this repo to GitHub Actions given Travis is unreliable.

@tjkandala
Copy link
Contributor

@felixfbecker I've migrated to GitHub Actions in #197, I'll rebase this PR onto master and merge + publish it.

@tjkandala tjkandala force-pushed the friendlier-language branch from e2a7de1 to 34108c8 Compare May 27, 2021 20:41
@@ -204,7 +204,7 @@ export const getBlameStatusBarItem = ({
)

return {
text: `Blame: ${username}${displayName}, ${distance}`,
text: `Author: ${username}${displayName}, ${distance}`,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Turns out we needed to update the tests too

@codecov-commenter
Copy link

Codecov Report

Merging #180 (cd8f76c) into master (814f186) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #180   +/-   ##
=======================================
  Coverage   47.58%   47.58%           
=======================================
  Files           4        4           
  Lines         124      124           
  Branches       25       25           
=======================================
  Hits           59       59           
  Misses         65       65           
Impacted Files Coverage Δ
src/blame.ts 84.37% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 814f186...cd8f76c. Read the comment docs.

@tjkandala tjkandala merged commit e265273 into master May 27, 2021
@felixfbecker felixfbecker deleted the friendlier-language branch May 27, 2021 20:48
@felixfbecker
Copy link
Contributor Author

Thank you ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants