Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Download from LGTM command requires a case-sensitive slug #892

Closed
aeisenberg opened this issue Jun 28, 2021 · 0 comments · Fixed by #978
Closed

Download from LGTM command requires a case-sensitive slug #892

aeisenberg opened this issue Jun 28, 2021 · 0 comments · Fixed by #978
Assignees
@mgsium
Labels
bug Something isn't working good first issue Good for newcomers VSCode

Comments

@aeisenberg
Copy link
Contributor

Describe the bug

Using the Download from LGTM command will fail if you enter a URL or slug that does not have the right case.

https://lgtm.com/projects/g/BOINC/boinc/

To reproduce

  1. Run Download from LGTM
  2. Enter the URL https://lgtm.com/projects/g/boinc/boinc/ (Notice that the correct URL is https://lgtm.com/projects/g/BOINC/boinc/)
  3. 404 error

Expected behavior
Should have downloaded and installed the database.

Additional context
It is likely that this is an LGTM issue where it is not able to locate repos case-insensitively. Suggestion from @p0:

For the /g/ provider, you could query github for the canonical case and pass it to LGTM (perhaps only after a 404).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mgsium mgsium

Labels
bug Something isn't working good first issue Good for newcomers VSCode
Projects
None yet
Milestone
No milestone
3 participants
@aeisenberg @adityasharad @mgsium