[PLUTO-1359] python/pylint installation#41
Conversation
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferences |
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for installing and running Python and Pylint by introducing new utilities for converting Pylint JSON output to SARIF, extracting Python runtime archives (with improved symlink handling), and updating configuration and CLI commands to manage Python and Pylint installations.
- Adds a new SARIF converter for Pylint JSON output in Go
- Updates the extraction process to defer symlink creation
- Enhances configuration, installation, and analysis workflows for Python and Pylint
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| utils/pylint_sarif_converter.go | Introduces conversion from Pylint JSON to SARIF |
| utils/extract.go | Adds deferred symlink handling and ensures parent directories are created |
| tools/pylintRunner.go | Implements command runner for Pylint analysis |
| config/runtime.go | Updates runtime information to support Python and Pylint |
| config/python-utils.go | Provides utilities for downloading and installing Python |
| config/pylint-utils.go | Adds functionality to install Pylint using the Python environment |
| cmd/install.go | Extends installation commands to include Python and Pylint |
| cmd/init.go | Updates configuration template to include Python and Pylint versions |
| cmd/analyze.go | Integrates Pylint analysis in the tooling |
| .codacy/codacy.yaml | Updates supported runtimes and tools in configuration |
Comments suppressed due to low confidence (1)
utils/extract.go:52
- Consider checking and handling the error returned by f.Open() instead of ignoring it.
stream, _ := f.Open()
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.