Skip to content
This repository has been archived by the owner on Aug 24, 2019. It is now read-only.

Fix JSXAttribute generation typo #13

Merged
merged 1 commit into from
Sep 15, 2018
Merged

Fix JSXAttribute generation typo #13

merged 1 commit into from
Sep 15, 2018

Conversation

TrySound
Copy link
Contributor

@TrySound TrySound commented Sep 7, 2018

No description provided.

@codecov
Copy link

codecov bot commented Sep 7, 2018

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   87.68%   87.68%           
=======================================
  Files          18       18           
  Lines         203      203           
  Branches       50       50           
=======================================
  Hits          178      178           
  Misses         20       20           
  Partials        5        5
Impacted Files Coverage Δ
packages/h2x-plugin-jsx/src/visitor.js 96.55% <100%> (ø) ⬆️
packages/h2x-plugin-jsx/src/generator.js 100% <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 ae03e3a...24a71cb. Read the comment docs.

@gregberge gregberge merged commit cc03a02 into smooth-code:master Sep 15, 2018
gregberge pushed a commit to gregberge/svgr that referenced this pull request Sep 16, 2018
In this diff I introduce the new option `svgProps` which is similar to
svgAttributes but applies values in interpolation style `{true}`.

This allows to specify expressions depending on template values for
example default `currentColor` in `fill` prop.

I mentioned in comment about later fixing api to smooth-code/h2x#13
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.

2 participants