Skip to content

Fixed CI has an error, update file format#1010

Open
qinwang-murphy wants to merge 1 commit into
sodafoundation:masterfrom
qinwang-murphy:wangqin_ci_20230828
Open

Fixed CI has an error, update file format#1010
qinwang-murphy wants to merge 1 commit into
sodafoundation:masterfrom
qinwang-murphy:wangqin_ci_20230828

Conversation

@qinwang-murphy

Copy link
Copy Markdown
Contributor

Fixed CI has an error, update file format

@qinwang-murphy

Copy link
Copy Markdown
Contributor Author

delfin/drivers/de11_emc/vmax/client.py:153:64: E231missing whitespace after ',
delfin/drivers/del1_emc/vmax/vmax.py:55:53: E231 missing whitespace after ','

@codecov

codecov Bot commented Aug 28, 2023

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.37%. Comparing base (978eff4) to head (5ffb838).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1010   +/-   ##
=======================================
  Coverage   70.37%   70.37%           
=======================================
  Files         195      195           
  Lines       24228    24228           
  Branches     3338     3338           
=======================================
  Hits        17051    17051           
  Misses       6016     6016           
  Partials     1161     1161           
Files with missing lines Coverage Δ
delfin/drivers/dell_emc/vmax/client.py 81.72% <100.00%> (ø)
delfin/drivers/dell_emc/vmax/vmax.py 83.33% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joseph-v joseph-v left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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