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

Feat: Rephrase for a more relaxed notification style, see #42 #43

Merged

Conversation

franz-josef-kaiser
Copy link

  • Removes the !!! shouting,
  • Removes unnecessary space usage,
  • Tries to to use Corectl instead of the corectl variant found here and there
  • Some rephrasing
  • Be nicer: Say please
  • Do not panic – we are dealing with developers (who do not read anyway) – when a VM goes down, then it goes dow
  • A little bit of grammar and such things here and there
  • Consolidates; Server is Off and Server is running into Server status: Off/Running
  • Replaces Active VMs: 0 with No Active VMs
  • … minor things

Hope you like it.

@franz-josef-kaiser franz-josef-kaiser changed the base branch from master to dev November 21, 2016 22:18
@franz-josef-kaiser
Copy link
Author

Puh … realized the wrong target branch (master) when I hit the "send" button. The target branch now is dev for a test run.

@rimusz
Copy link
Member

rimusz commented Nov 22, 2016

awesome, thanks. will test it out next when i'm back from holiday:)

@rimusz rimusz merged commit 6fb30ae into TheNewNormal:dev Nov 23, 2016
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.

2 participants