Skip to content

Comments

test: update format_config script#3473

Merged
dl239 merged 1 commit into4paradigm:mainfrom
lqy222:update_test_script
Aug 30, 2023
Merged

test: update format_config script#3473
dl239 merged 1 commit into4paradigm:mainfrom
lqy222:update_test_script

Conversation

@lqy222
Copy link
Collaborator

@lqy222 lqy222 commented Aug 29, 2023

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

remove format_config.sh sudo request

  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)?

@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01% ⚠️

Comparison is base (1ea8e58) 75.37% compared to head (b3775be) 75.36%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3473      +/-   ##
============================================
- Coverage     75.37%   75.36%   -0.01%     
  Complexity      414      414              
============================================
  Files           707      707              
  Lines        128561   128561              
  Branches       1191     1191              
============================================
- Hits          96906    96895      -11     
- Misses        31414    31425      +11     
  Partials        241      241              

see 17 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@tobegit3hub tobegit3hub left a comment

Choose a reason for hiding this comment

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

LGTM

@dl239 dl239 merged commit badf7ab into 4paradigm:main Aug 30, 2023
dl239 pushed a commit that referenced this pull request Sep 19, 2023
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