Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

vagrant-notify 0.5.0 #27

Merged
merged 16 commits into from
Jun 22, 2016
Merged

vagrant-notify 0.5.0 #27

merged 16 commits into from
Jun 22, 2016

Conversation

alpha01
Copy link
Collaborator

@alpha01 alpha01 commented May 11, 2016

As per #24 (comment) here is the PR 馃嵒

IMPROVEMENTS

  • Notification server runs as a daemon (fork() has been removed so it's compatible with Windows). [GH-14]
  • Windows support (beta) [GH-14]
  • Notification server is accessible only from localhost.
  • Notification server information is displayed each time vagrant is started and halted.
  • New plugin command. Ability to view notification server status, and ability for manual restart. CLI
  • New notify-send example scripts for OS X and Windows

BUG FIXES

  • vagrant suspend/resume stops/starts notification server respectively. [GH-18]
  • Plugin causes VM boot to fail. [GH-22]

@alpha01
Copy link
Collaborator Author

alpha01 commented May 11, 2016

I think Travis CI needs to be updated so the tests run on newer version of Ruby.

@fgrehm
Copy link
Owner

fgrehm commented May 24, 2016

Thanks for the PR and welcome aboard! 馃嵒
LMK when u are ready to cut a release and send me you rubygems email so I can add you there 馃槃

@alpha01
Copy link
Collaborator Author

alpha01 commented May 24, 2016

My rubygems email is root@rubyninja.org. This commit should be ready for release. I guess the only thing left if just updating the CI so the tests run on newer versions of Ruby. 馃嵒

@fgrehm
Copy link
Owner

fgrehm commented Jun 8, 2016

Cool! Please ping me when you get to a green build and I'll hook u up with rubygems access!

@alpha01
Copy link
Collaborator Author

alpha01 commented Jun 13, 2016

Alright, CI tests should be green. (Will be green once it's merged since a new build needs to be run as .travis.yml was modified). So now, all is needed my rubygems account root@rubyninja.org to have access to the project's gem, and I should be ready to cut a new release.

@eli007s
Copy link

eli007s commented Jun 20, 2016

whats the status on this PR?

@alpha01
Copy link
Collaborator Author

alpha01 commented Jun 20, 2016

I'm just waiting for @fgrehm to grant access to my rubygems account so account cut over a new release..

@fgrehm
Copy link
Owner

fgrehm commented Jun 22, 2016

@alpha01 done!

@alpha01 alpha01 merged commit 1a9ca8c into fgrehm:master Jun 22, 2016
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.

None yet

3 participants