Skip to content

removing unsed import in registry#494

Merged
thinkingserious merged 1 commit into
sendgrid:masterfrom
gardlt:removing-unsed-import
Oct 30, 2017
Merged

removing unsed import in registry#494
thinkingserious merged 1 commit into
sendgrid:masterfrom
gardlt:removing-unsed-import

Conversation

@gardlt

@gardlt gardlt commented Oct 29, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • os import is not used

Fixes

Checklist

  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the [Contribution Guide] and my PR follows them.
  • I updated my branch with the master branch.
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation about the functionality in the appropriate .md file
  • I have added inline documentation to the code I modified

Short description of what this PR does:

  • removes unused import

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

@gardlt
removing unsed import in registry
3267c94 
- os import is not used
@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Oct 29, 2017
@SendGridDX

SendGridDX commented Oct 29, 2017 •
edited
Loading

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@codecov-io

codecov-io commented Oct 29, 2017 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #494 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #494   +/-   ##
=====================================
  Coverage      83%    83%           
=====================================
  Files          30     30           
  Lines        1024   1024           
  Branches      160    160           
=====================================
  Hits          850    850           
  Misses         84     84           
  Partials       90     90

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 a8b70c5...3267c94. Read the comment docs.

@thinkingserious thinkingserious added difficulty: easy fix is easy in difficulty hacktoberfest labels Oct 30, 2017
@thinkingserious
thinkingserious merged commit b072072 into sendgrid:master Oct 30, 2017
@thinkingserious

thinkingserious commented Oct 30, 2017

Copy link
Copy Markdown
Contributor

Hello @gardlt,

Thanks again for the PR!

We want to show our appreciation by sending you some swag. Could you please fill out this form so we can send it to you? Thanks!

Team SendGrid DX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

difficulty: easy fix is easy in difficulty status: code review request requesting a community code review or review from Twilio

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gardlt @SendGridDX @codecov-io @thinkingserious