Skip to content

feat: manual ripgrep downloads for JB #5582

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.

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

Merged
merged 5 commits into from
May 9, 2025
Merged

Conversation

Patrick-Erichsen
Copy link
Collaborator

@Patrick-Erichsen Patrick-Erichsen commented May 9, 2025

Description

Resolves #5542

This PR introduces a manual step in the build process for JetBrains to install ripgrep for the respective OS/arch.

In our VS Code build process, we don't need to do this because we run builds in CI on the appropriate runners for each OS/arch. But for JB, we need to upload a single zip that contains all the required deps for each OS/arch, so we only run on a single runner (macOS-latest), which then causes us to only install the ripgrep binary for that platform.

@Patrick-Erichsen Patrick-Erichsen requested a review from a team as a code owner May 9, 2025 01:50
@Patrick-Erichsen Patrick-Erichsen requested review from sestinj and removed request for a team May 9, 2025 01:50
Copy link

netlify bot commented May 9, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit e37c709
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/681e5d1d7edce00008ed4acb

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label May 9, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels May 9, 2025
@Patrick-Erichsen
Copy link
Collaborator Author

@uinstinct @fbricon - mind taking a look at this since you both have some context from the recent ripgrep PR.

Copy link
Contributor

@uinstinct uinstinct left a comment

Choose a reason for hiding this comment

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

Tested and Ripgrep works fine on both vscode and jetbrains (intellij community)!

image

image

(BTW like the new PR size label feature!)

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 9, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit 6953fc8 into main May 9, 2025
34 checks passed
@Patrick-Erichsen Patrick-Erichsen deleted the pe/ripgrep-download branch May 9, 2025 20:13
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Error executing ripgrep tool on PyCharm (Windows)
3 participants