Skip to content

Comments

chore: updated the RL wrapper installation path#917

Merged
pmathew92 merged 1 commit intomainfrom
rl_changes
Feb 19, 2026
Merged

chore: updated the RL wrapper installation path#917
pmathew92 merged 1 commit intomainfrom
rl_changes

Conversation

@pmathew92
Copy link
Contributor

Changes

GH action pipeline change for RL scanner

Copilot AI review requested due to automatic review settings February 19, 2026 07:12
@pmathew92 pmathew92 requested a review from a team as a code owner February 19, 2026 07:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the RL (Reversing Labs) wrapper installation configuration in the GitHub Actions workflow. The change modifies how the rl-wrapper Python package is installed from a JFrog Artifactory repository by updating the index URL path and removing the version constraint.

Changes:

  • Updated the pip install command for rl-wrapper to use a different Artifactory repository path
  • Removed the version constraint (>=1.0.0) from the rl-wrapper installation
Comments suppressed due to low confidence (1)

.github/actions/rl-scanner/action.yml:36

  • Removing the version constraint (>=1.0.0) from the rl-wrapper installation could lead to dependency management issues. Without a version constraint, pip will always install the latest version, which could introduce breaking changes or incompatibilities without warning. Consider keeping at least a major version constraint to ensure stability and reproducibility of the build pipeline.
        pip install rl-wrapper --index-url "https://${{ env.PRODSEC_TOOLS_USER }}:${{ env.PRODSEC_TOOLS_TOKEN }}@a0us.jfrog.io/artifactory/api/pypi/python/simple"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pmathew92 pmathew92 merged commit 7b5611a into main Feb 19, 2026
12 checks passed
@pmathew92 pmathew92 deleted the rl_changes branch February 19, 2026 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants