Skip to content

[EASY] Code Cleanup#310

Merged
Bento007 merged 2 commits intomasterfrom
tsmith-cleanup2
Nov 1, 2019
Merged

[EASY] Code Cleanup#310
Bento007 merged 2 commits intomasterfrom
tsmith-cleanup2

Conversation

@Bento007
Copy link
Copy Markdown
Collaborator

@Bento007 Bento007 commented Nov 1, 2019

  • Fixing batch reference name. The name used was incorrect.
  • Fixing batch_write retries. 'allowed_errors' didn't match due to an extra ':' at the end of the parsed error from the batch_write call.
  • Correcting typing
  • Cleaning up swagger spacing
  • Removing unused code

- Fixing batch reference name. The name used was incorrect.
- Fixing batch_write retries. 'allowed_errors' didn't match due to an extra ':' at the end of the parsed error from the batch_write call.
- Correcting typing
- Cleaning up swagger spacing
- Removing unused code
@Bento007 Bento007 requested a review from chmreid November 1, 2019 20:52
@Bento007 Bento007 self-assigned this Nov 1, 2019
Copy link
Copy Markdown
Contributor

@DailyDreaming DailyDreaming left a comment

Choose a reason for hiding this comment

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

LGTM.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #310 into master will decrease coverage by 0.5%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #310      +/-   ##
==========================================
- Coverage   89.81%   89.31%   -0.51%     
==========================================
  Files          18       18              
  Lines        1660     1657       -3     
==========================================
- Hits         1491     1480      -11     
- Misses        169      177       +8
Impacted Files Coverage Δ
fusillade/clouddirectory.py 93.93% <50%> (-0.03%) ⬇️
fusillade/utils/retry.py 75.72% <0%> (-7.77%) ⬇️

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 802898c...78df575. Read the comment docs.

@Bento007 Bento007 merged commit c07a7d4 into master Nov 1, 2019
@Bento007 Bento007 deleted the tsmith-cleanup2 branch November 1, 2019 21:04
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.

4 participants