Skip to content
This repository has been archived by the owner on Nov 7, 2023. It is now read-only.

Remove unused imports #90

Merged
merged 1 commit into from
Oct 19, 2020
Merged

Remove unused imports #90

merged 1 commit into from
Oct 19, 2020

Conversation

jessgusclark
Copy link
Member

Removed unused imports reported by LGTM.

  • I left shim.js and metro.js files alone. These unused imports will remain.
  • I commented one line out in the Verifier that references code that is commented out below it for using the app with an emulator. I removed the "unused" imports in the comment section. I'm not sure about this one.

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2020

This pull request fixes 10 alerts when merging 116cc1f into daf8415 - view on LGTM.com

fixed alerts:

  • 10 for Unused variable, import, function or class

ilanolkies
ilanolkies approved these changes Oct 19, 2020
View reviewed changes
Copy link
Contributor

@ilanolkies ilanolkies left a comment

Choose a reason for hiding this comment

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

Great!

@ilanolkies ilanolkies merged commit 1214520 into develop Oct 19, 2020
@delete-merged-branch delete-merged-branch bot deleted the code-qa branch October 19, 2020 20:10
@ilanolkies ilanolkies mentioned this pull request Oct 22, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ilanolkies ilanolkies ilanolkies approved these changes

@javiesses javiesses Awaiting requested review from javiesses

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jessgusclark @ilanolkies