Skip to content

[PLUTO-1359] python/pylint installation#41

Closed
zhamborova wants to merge 2 commits intomainfrom
PLUTO-1359_pylint_tool
Closed

[PLUTO-1359] python/pylint installation#41
zhamborova wants to merge 2 commits intomainfrom
PLUTO-1359_pylint_tool

Conversation

@zhamborova
Copy link
Copy Markdown
Contributor

No description provided.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Mar 28, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-3.06% 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (36db748) 961 110 11.45%
Head commit (c38a9d2) 1312 (+351) 110 (+0) 8.38% (-3.06%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#41) 363 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@zhamborova zhamborova requested a review from Copilot March 28, 2025 15:55
Copy link
Copy Markdown
Contributor

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 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()

Comment thread config/pylint-utils.go Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@zhamborova zhamborova closed this Apr 1, 2025
@zhamborova zhamborova deleted the PLUTO-1359_pylint_tool branch April 1, 2025 08:53
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.

2 participants