Skip to content

🐛 Fix Golang analysis issues on Windows#1428

Open
abrugaro wants to merge 5 commits into
konveyor:mainfrom
abrugaro:1427-go-analysis
Open

🐛 Fix Golang analysis issues on Windows#1428
abrugaro wants to merge 5 commits into
konveyor:mainfrom
abrugaro:1427-go-analysis

Conversation

@abrugaro

@abrugaro abrugaro commented May 28, 2026

Copy link
Copy Markdown
Contributor

Resolves #1427
Resolves #1429

Needs konveyor/analyzer-lsp#1157 & konveyor/analyzer-lsp#1159
to work.

This PR:

  • Updates how the extension was handling file uris on Windows to correctly report analysis incidents.
  • Allows the JS provider to initialize even when there is a version mismatch, this is the current behaviour of the Go and Java providers

abrugaro added 4 commits May 28, 2026 12:00
Signed-off-by: Alejandro Brugarolas <abrugaro@redhat.com>
Signed-off-by: Alejandro Brugarolas <abrugaro@redhat.com>
Signed-off-by: Alejandro Brugarolas <abrugaro@redhat.com>
@abrugaro abrugaro self-assigned this May 28, 2026
@abrugaro abrugaro requested a review from a team as a code owner May 28, 2026 12:26
@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@abrugaro, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 39 minutes and 3 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4264cb34-9e1d-49c0-8f09-1151d88720dc

📥 Commits

Reviewing files that changed from the base of the PR and between f865f9f and 398911e.

📒 Files selected for processing (4)
  • changes/unreleased/1427-fix-windows-lsp-proxy-uri-encoding.yaml
  • vscode/go/src/goVscodeProxyServer.ts
  • vscode/javascript/src/extension.ts
  • vscode/javascript/src/vscodeProxyServer.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Signed-off-by: Alejandro Brugarolas <abrugaro@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

1 participant