Skip to content

refactor: Remove redundant open mode argument#538

Merged
gfieni merged 2 commits intomasterfrom
refactor/remove-redundant-open-mode
Feb 19, 2025
Merged

refactor: Remove redundant open mode argument#538
gfieni merged 2 commits intomasterfrom
refactor/remove-redundant-open-mode

Conversation

@gfieni
Copy link
Contributor

@gfieni gfieni commented Feb 19, 2025

No description provided.

@gfieni gfieni self-assigned this Feb 19, 2025
@gfieni gfieni requested a review from a team as a code owner February 19, 2025 11:06
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Feb 19, 2025

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 67.92%. Comparing base (b620913) to head (8ab74aa).
Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/powerapi/database/csvdb.py 0.00% 1 Missing ⚠️
src/powerapi/database/file_db.py 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #538   +/-   ##
=======================================
  Coverage   67.92%   67.92%           
=======================================
  Files          71       71           
  Lines        2965     2965           
=======================================
  Hits         2014     2014           
  Misses        951      951           
Files with missing lines Coverage Δ
src/powerapi/cli/config_parser.py 91.26% <100.00%> (ø)
src/powerapi/database/csvdb.py 32.75% <0.00%> (ø)
src/powerapi/database/file_db.py 46.66% <0.00%> (ø)

@gfieni gfieni merged commit 6cc7b67 into master Feb 19, 2025
10 of 11 checks passed
@gfieni gfieni deleted the refactor/remove-redundant-open-mode branch February 19, 2025 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant