Skip to content

Commit

Permalink
feat: Fetch log per-revision rather than on blame (#488)
Browse files Browse the repository at this point in the history
Addresses #91 and #481

* Fetches a log when you click on a line
* Stores log data independent of blame
* Maps data and updates decoration as required when line is clicked

Further optimisations to be made:
* Store log data at a repository level, so that they can be shared
across files

## Demo
https://github.com/BeauAgst/svn-blamer/assets/10343831/ecf8ac51-e15d-4b3d-9fdd-c0f8b57c40e0
  • Loading branch information
metacurb authored Apr 2, 2024
1 parent ee20110 commit 5aa65c1
Show file tree
Hide file tree
Showing 14 changed files with 247 additions and 173 deletions.
12 changes: 6 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,18 +32,18 @@ This extension contributes the following commands to the Command palette.

## Configuration

| Setting | Description | Default value |
| ---------------------------- | ------------------------------------------------------------------------------------------ | ------------- |
| **Auto Blame** | Automatically blames files as you open them. | `true` |
| **Enable Details** | Enables popup revision log data. Disabling this setting will significantly speed up blame. | `false` |
| **Enable Visual Indicators** | Toggle visual indicators that sit to the left of the line number. | `true` |
| Setting | Description | Default value |
| ---------------------------- | ----------------------------------------------------------------- | ------------- |
| **Auto Blame** | Automatically blames files as you open them. | `true` |
| **Enable Details** | Fetches and displays revision log data in the popup. | `false` |
| **Enable Visual Indicators** | Toggle visual indicators that sit to the left of the line number. | `true` |

## Known Issues

- Causes slowdown when the "**Enable Details**" setting is enabled because all unique logs have to be retrieved first.
- Authentication errors [#5](https://github.com/BeauAgst/svn-blamer/issues/5)

## Feedback & Contributing

Please report any bugs, suggestions or documentation requests via [issues](https://github.com/BeauAgst/svn-blamer/issues)

Feel free to submit [pull requests](https://github.com/BeauAgst/svn-blamer/pulls)
20 changes: 18 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@
"watch": "npm run esbuild -- --sourcemap --watch"
},
"devDependencies": {
"@types/lodash.merge": "^4.6.9",
"@types/luxon": "^3.4.2",
"@types/mocha": "^10.0.6",
"@types/node": "18.x",
Expand All @@ -111,6 +112,7 @@
"typescript": "^5.4.3"
},
"dependencies": {
"lodash.merge": "^4.6.2",
"luxon": "^3.4.4",
"xml-js": "^1.6.11"
},
Expand All @@ -120,4 +122,4 @@
"prettier --cache --write --ignore-unknown"
]
}
}
}
Loading

0 comments on commit 5aa65c1

Please sign in to comment.