Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Convert SearchBar to es6 - #2595

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
AnshulMalik:es6
Apr 8, 2017
Merged

Convert SearchBar to es6#2595
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
AnshulMalik:es6

Conversation

@AnshulMalik

Copy link
Copy Markdown
Contributor

Associated Issue: #1890

Summary of Changes

  • Converted SearchBar to es6 class

@codecov

codecov Bot commented Apr 8, 2017

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2595   +/-   ##
=======================================
  Coverage   57.76%   57.76%           
=======================================
  Files          51       51           
  Lines        1996     1996           
  Branches      401      401           
=======================================
  Hits         1153     1153           
  Misses        843      843

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 07edbdc...9118dda. Read the comment docs.

Comment thread src/components/Editor/SearchBar.js Outdated
import { isEnabled } from "devtools-config";
import { filter } from "fuzzaldrin-plus";
import Svg from "../shared/Svg";
import { default as actions } from "../../actions";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we replace this with import actions from "../../actions"; ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, Nice catch. Thanks, I'll do a quick change.

@jasonLaster

Copy link
Copy Markdown
Contributor

Awesome!

@jasonLaster
jasonLaster merged commit 6906d86 into firefox-devtools:master Apr 8, 2017
@AnshulMalik
AnshulMalik deleted the es6 branch April 8, 2017 14:56
DanUgelow pushed a commit to DanUgelow/debugger.html that referenced this pull request May 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants