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

Fix syntax error in Redirect component #21

Merged
merged 1 commit into from
May 9, 2019
Merged

Fix syntax error in Redirect component #21

merged 1 commit into from
May 9, 2019

Conversation

alex-tolkunov
Copy link

No description provided.

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   94.57%   94.57%           
=======================================
  Files           5        5           
  Lines         129      129           
  Branches       26       26           
=======================================
  Hits          122      122           
  Misses          6        6           
  Partials        1        1
Impacted Files Coverage Δ
index.js 91.22% <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 44a15e3...3711143. Read the comment docs.

@molefrog
Copy link
Owner

molefrog commented May 9, 2019

Thanks for that! I'm going to release a new version with a bugfix.

@molefrog molefrog merged commit 0e516cd into molefrog:master May 9, 2019
@molefrog
Copy link
Owner

@StrayFromThePath Thanks again for fixing this one! I've just released a new version v1.2.1 with the bugfix. Two things I also did in order to prevent errors like this from happening:

Great job!

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