REditorSupport / REditorSupport/vscode-R

Lintr errors whilst trying to lint staged / indexed files in git

Open
#1,436 2 comments 3 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
TypeScript
Stars
1.2k
Forks
139
Avg merge
2h 46m
Merged PRs (30d)
5

Description

Description of issue

It looks like the lintr is trying to execute on staged/indexed files in git but just errors when trying to do so. Worse still is that these errors end up littering the Problems pane and are never cleared up. Only solution seems to be to restart vscode

Gif of issue

2023-10-11 11 42 06

Versions
MacOS - 13.3.1 (22E261)
REditorSupport.r - 2.8.2
R - 4.3.1
lintr - 3.1.0
language server - 0.3.16
Settings file
	"files.associations": {
		"*.rmd": "rmd"
	},
    "r.alwaysUseActiveTerminal": true,
    "r.liveShare.defaults.shareWorkspace": false,
    "r.lsp.promptToInstall": false,
    "r.session.objectTimeout": 10,
    "r.session.useWebServer": false,
    "r.sessionWatcher": true,

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file or test is named. Start by reproducing the staged/indexed-file linting failure in VS Code using the listed extension, R, and lintr versions, then trace the extension's linting and Problems-pane integration. Done means the lint operation no longer errors and stale diagnostics are cleared without restarting VS Code.

Written by the indexing model from the issue text.

Assessment

Tech stack
r, typescript, vscode
Domain
developer-experience, tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
32/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.