Skip to content

Delete unused variable#4353

Merged
justindho merged 5 commits intoaws:developfrom
yoshiken:delete_unused_variable
May 25, 2022
Merged

Delete unused variable#4353
justindho merged 5 commits intoaws:developfrom
yoshiken:delete_unused_variable

Conversation

@yoshiken
Copy link
Copy Markdown
Contributor

Removed unused variables.

This will lead to a slight performance improvement.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 26, 2019

Codecov Report

Merging #4353 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop   #4353      +/-   ##
==========================================
- Coverage     94.5%   94.5%   -0.01%     
==========================================
  Files          188     188              
  Lines        14166   14158       -8     
==========================================
- Hits         13388   13380       -8     
  Misses         778     778
Impacted Files Coverage Δ
awscli/customizations/s3/utils.py 97.96% <ø> (-0.01%) ⬇️
awscli/customizations/emr/instancefleetsutils.py 100% <ø> (ø) ⬆️
awscli/customizations/emr/describecluster.py 93.02% <ø> (-0.6%) ⬇️
awscli/customizations/toplevelbool.py 100% <100%> (ø) ⬆️
awscli/clidocs.py 99.14% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60dded0...49c7fb4. Read the comment docs.

@kdaily kdaily added the pr:needs-review This PR needs a review from a Member. label Mar 2, 2021
yoshiken added 5 commits May 25, 2022 12:46
This loop does not use key in dict.
This loop does not use key in dict.
This variable is not used, And can reduce the else statement.
This variable is not used, And can reduce the if statement.
This variable is not used, And can reduce the if statement and for 
statement.
@justindho justindho force-pushed the delete_unused_variable branch from 49c7fb4 to 563a75b Compare May 25, 2022 19:47
@justindho justindho added community ready-for-review and removed pr:needs-review This PR needs a review from a Member. labels May 25, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 25, 2022

Codecov Report

Merging #4353 (563a75b) into develop (f2f8cce) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #4353      +/-   ##
===========================================
- Coverage    92.88%   92.87%   -0.01%     
===========================================
  Files          204      204              
  Lines        16336    16329       -7     
===========================================
- Hits         15173    15166       -7     
  Misses        1163     1163              
Impacted Files Coverage Δ
awscli/customizations/emr/describecluster.py 93.02% <ø> (-0.60%) ⬇️
awscli/customizations/emr/instancefleetsutils.py 100.00% <ø> (ø)
awscli/clidocs.py 99.19% <100.00%> (ø)
awscli/customizations/toplevelbool.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2f8cce...563a75b. Read the comment docs.

Copy link
Copy Markdown
Contributor

@justindho justindho left a comment

Choose a reason for hiding this comment

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

Thanks, @yoshiken!

@justindho justindho merged commit 20348b2 into aws:develop May 25, 2022
@justindho justindho added pending-release This issue will be fixed by an approved PR that hasn't been released yet. released and removed ready-for-review pending-release This issue will be fixed by an approved PR that hasn't been released yet. labels May 25, 2022
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.

6 participants