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

Interpolation for multiple occurances #4761

Merged
merged 1 commit into from
Nov 9, 2018

Conversation

lflpowell
Copy link
Contributor

Issue:

What I did

Replace a single occurrence string replace with a regular expression for global replacement

How to test

  • Is this testable with Jest or Chromatic screenshots?
    Yes

  • Does this need a new example in the kitchen sink apps?
    No

  • Does this need an update to the documentation?
    No

@codecov
Copy link

codecov bot commented Nov 9, 2018

Codecov Report

Merging #4761 into next will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##             next    #4761   +/-   ##
=======================================
  Coverage   35.61%   35.61%           
=======================================
  Files         557      557           
  Lines        6776     6776           
  Branches      904      904           
=======================================
  Hits         2413     2413           
  Misses       3893     3893           
  Partials      470      470
Impacted Files Coverage Δ
lib/core/src/server/utils.js 36.36% <100%> (ø) ⬆️

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 ba8680f...8caf4bc. Read the comment docs.

@ndelangen ndelangen added this to the next milestone Nov 9, 2018
@ndelangen ndelangen merged commit cc6cc4a into storybookjs:next Nov 9, 2018
@ndelangen
Copy link
Member

Awesome! Thanks for fixing this @lflpowell

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

Successfully merging this pull request may close these issues.

2 participants