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 #156 #183

Merged
merged 1 commit into from
Nov 12, 2018
Merged

Fix #156 #183

merged 1 commit into from
Nov 12, 2018

Conversation

Fi1osof
Copy link
Contributor

@Fi1osof Fi1osof commented Nov 9, 2018

No description provided.

@Fi1osof
Copy link
Contributor Author

Fi1osof commented Nov 12, 2018

@cfnelson please, merge PR. It's critical for me. Thanks!

@cfnelson cfnelson merged commit 37bece6 into Urigo:master Nov 12, 2018
@Fi1osof
Copy link
Contributor Author

Fi1osof commented Nov 12, 2018

@cfnelson thanks!

@cfnelson
Copy link
Contributor

On mobile atm, will publish a npm release once I am back at a keyboard. Will be a few hours.

@cfnelson
Copy link
Contributor

Should be able to use the master branch directly until then. Or if you wish to make a pr for a new release. I'll accept that and deploy the new version.

@Fi1osof
Copy link
Contributor Author

Fi1osof commented Nov 13, 2018

@cfnelson I do not quite understand, should I do something else?

@RichardLitt
Copy link
Contributor

@Fi1osof I don't think so; I think that @cfnelson was just letting you know what you could use before he makes a new release.

@cfnelson Have you published yet? :)

@cfnelson cfnelson mentioned this pull request Nov 14, 2018
@cfnelson
Copy link
Contributor

@Fi1osof Sorry about that, was asking if you'd bump the pkg version & CHANGELOG.md. Sorry for not being clear.
I've been flat out & haven't had any time 😅 . It's published now and hopefully it fixes everything for you! 🙌 Bumped npm pkgs as well to fix reported security vulnerabilities. Release v1.5.8
cc @RichardLitt the above ^

@Fi1osof
Copy link
Contributor Author

Fi1osof commented Nov 14, 2018

@RichardLitt , @cfnelson ok, i got it. Thnaks. Will try next time.

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.

3 participants