Open
Conversation
Contributor
Qodana for Ruby155 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
mridang
force-pushed
the
main
branch
3 times, most recently
from
ea75fc5 to
a6c0c03
Compare
mridang
force-pushed
the
feat/implement-eap-qodana
branch
from
6334083 to
f687fc3
Compare
mridang
changed the title
mridang
force-pushed
the
feat/implement-eap-qodana
branch
from
616da58 to
44e406d
Compare
mridang
force-pushed
the
feat/implement-eap-qodana
branch
from
44e406d to
239b7cb
Compare
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s Qodana setup to use the Ruby linter (EAP), adds a Bundler bootstrap step/config, and enables using an existing SARIF baseline in the GitHub Actions workflow.
Changes:
- Switch Qodana linter image to
jetbrains/qodana-ruby:2025.3-eap-ruby3.4-privilegedand addbundle installbootstrap inqodana.yaml. - Enable Qodana baseline usage in
.github/workflows/qodana.yml. - Add
.bundle/config(Bundler install path) and ignoreGemfile.lock.
Reviewed changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
qodana.yaml |
Moves Qodana to the Ruby EAP linter and adds a Bundler bootstrap step. |
.github/workflows/qodana.yml |
Passes the SARIF baseline to Qodana action. |
.bundle/config |
Configures Bundler to install gems under vendor/bundle. |
.gitignore |
Adds Gemfile.lock to ignored files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The .idea/misc.xml and client-ruby.iml reference local Ruby SDK names (ruby-2.6.10-p210, ruby-3.0.6-p216) that don't exist inside the Qodana Docker container, causing all 155 inspection errors (RubyInterpreter, RubyResolve, GemInspection). Removing them in bootstrap lets Qodana auto-detect the container's Ruby 3.4.8 interpreter.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Documentation:
Checklist: