Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #1041:Removed unnecessary imports and unused variables #1042

Merged
merged 1 commit into from Jun 22, 2018

Conversation

praveenojha33
Copy link
Member

@praveenojha33 praveenojha33 commented Jun 22, 2018

Fixes #1041

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream master branch.
  • I have added necessary documentation (if appropriate)
  • Added Surge preview link

Changes proposed in this pull request:

Removed unnecessary and unused variables to optimize app.
Surge Link:https://pr-1042-fossasia-susper.surge.sh

@praveenojha33 praveenojha33 changed the title Fixes #1041:Removed unnecessary and unused variables Fixes #1041:Removed unnecessary imports and unused variables Jun 22, 2018
@codecov
Copy link

codecov bot commented Jun 22, 2018

Codecov Report

Merging #1042 into master will decrease coverage by <.01%.
The diff coverage is 92.59%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1042      +/-   ##
=========================================
- Coverage    55.4%   55.4%   -0.01%     
=========================================
  Files          47      47              
  Lines        1193    1175      -18     
  Branches      146     144       -2     
=========================================
- Hits          661     651      -10     
+ Misses        444     439       -5     
+ Partials       88      85       -3
Impacted Files Coverage Δ
src/app/speechtotext/speechtotext.component.ts 41.42% <ø> (ø) ⬆️
src/app/about/about.component.ts 100% <ø> (ø) ⬆️
src/app/statsbox/statsbox.component.ts 61.97% <ø> (ø) ⬆️
src/app/utils.ts 66.66% <ø> (ø) ⬆️
src/app/index/index.component.ts 81.25% <ø> (ø) ⬆️
src/app/crawlstart/crawlstart.component.ts 46.66% <100%> (ø) ⬆️
src/app/searchsettings/searchsettings.component.ts 61.11% <100%> (ø) ⬆️
src/app/contact/contact.component.ts 55% <100%> (ø) ⬆️
src/app/app.component.ts 63.63% <100%> (ø) ⬆️
src/app/auto-complete/auto-complete.component.ts 42.1% <100%> (ø) ⬆️
... and 6 more

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 530ba21...49ba28c. Read the comment docs.

@@ -1,10 +1,8 @@
import { Injectable } from '@angular/core';
import { Location } from '@angular/common';
import {Effect, Actions, toPayload} from '@ngrx/effects';
import {Effect, Actions } from '@ngrx/effects';
Copy link
Member

Choose a reason for hiding this comment

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

You are organising imports, please create space before Effect.

import { async, ComponentFixture, TestBed, fakeAsync } from '@angular/core/testing';
import { Component, Input, Output } from '@angular/core';

import { async, ComponentFixture, TestBed} from '@angular/core/testing';
Copy link
Member

Choose a reason for hiding this comment

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

You are organising imports, please create space before TestBed.

Copy link
Member

@simsausaurabh simsausaurabh left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mariobehling mariobehling merged commit f4a091e into fossasia:master Jun 22, 2018
@praveenojha33 praveenojha33 deleted the video branch June 23, 2018 01:01
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.

None yet

3 participants