Skip to content

Refactor docker images in EVerest

Codacy Production / Codacy Static Code Analysis required action Aug 8, 2024 in 0s

27 new issues (0 max.) of at least minor severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 27
           

Complexity increasing per file
==============================
- everest_dev_tool/src/everest_dev_tool/parser.py  2
- everest_dev_tool/src/everest_dev_tool/services.py  5
- everest_dev_tool/src/everest_dev_tool/git_handlers.py  2
- everest_dev_tool/src/everest_dev_tool/__init__.py  1

See the complete overview on Codacy

Annotations

Check notice on line 11 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L11 

Expected: 80; Actual: 126

Check notice on line 15 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L15 

Expected: 80; Actual: 110

Check notice on line 31 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L31 

Expected: 80; Actual: 101

Check notice on line 31 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L31 

Lists should be surrounded by blank lines

Check notice on line 32 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L32 

Expected: 80; Actual: 98

Check notice on line 42 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L42 

Expected: 80; Actual: 103

Check notice on line 46 in README.md

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

README.md#L46 

Expected: 80; Actual: 136

Check warning on line 17 in devcontainer/setup-devcontainer.sh

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

devcontainer/setup-devcontainer.sh#L17 

Double quote to prevent globbing and word splitting.

Check warning on line 19 in devcontainer/setup-devcontainer.sh

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

devcontainer/setup-devcontainer.sh#L19 

Double quote to prevent globbing and word splitting.

Check warning on line 36 in devcontainer/setup-devcontainer.sh

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

devcontainer/setup-devcontainer.sh#L36 

Double quote to prevent globbing and word splitting.

Check failure on line 5 in devcontainer/template/.devcontainer/general-devcontainer/Dockerfile

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

devcontainer/template/.devcontainer/general-devcontainer/Dockerfile#L5 

Do not use sudo as it leads to unpredictable behavior. Use a tool like gosu to enforce root

Check notice on line 15 in devcontainer/template/.devcontainer/general-devcontainer/Dockerfile

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

devcontainer/template/.devcontainer/general-devcontainer/Dockerfile#L15 

Double quote to prevent globbing and word splitting.

Check notice on line 17 in devcontainer/template/.devcontainer/general-devcontainer/Dockerfile

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

devcontainer/template/.devcontainer/general-devcontainer/Dockerfile#L17 

Double quote to prevent globbing and word splitting.

Check warning on line 3 in everest_dev_tool/src/everest_dev_tool/git_handlers.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/git_handlers.py#L3 

Consider possible security implications associated with the subprocess module.

Check notice on line 11 in everest_dev_tool/src/everest_dev_tool/git_handlers.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/git_handlers.py#L11 

Using an f-string that does not have any interpolated variables (f-string-without-interpolation)

Check notice on line 13 in everest_dev_tool/src/everest_dev_tool/git_handlers.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/git_handlers.py#L13 

Using an f-string that does not have any interpolated variables (f-string-without-interpolation)

Check warning on line 4 in everest_dev_tool/src/everest_dev_tool/parser.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/parser.py#L4 

Unused import debug_handlers

Check notice on line 4 in everest_dev_tool/src/everest_dev_tool/parser.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/parser.py#L4 

Unused import debug_handlers (unused-import)

Check notice on line 1 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L1 

/src/everest_dev_tool/src/everest_dev_tool/services.py: Fatal error while checking '/src/everest_dev_tool/src/everest_dev_tool/services.py'. Please open an issue in our bug tracker so we address this. There is a pre-filled template that you can use in '/home/docker/.cache/pylint/pylint-crash-2024-08-08-07-06-34.txt'. (astroid-error)

Check warning on line 4 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L4 

Consider possible security implications associated with the subprocess module.

Check warning on line 51 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L51 

subprocess call - check for execution of untrusted input.

Check notice on line 67 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L67 

Trailing whitespace

Check warning on line 79 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L79 

subprocess call - check for execution of untrusted input.

Check warning on line 174 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L174 

subprocess call - check for execution of untrusted input.

Check warning on line 188 in everest_dev_tool/src/everest_dev_tool/services.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

everest_dev_tool/src/everest_dev_tool/services.py#L188 

subprocess call - check for execution of untrusted input.
View more details on Codacy Production