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(setup): enable systemd service during setup #249

Closed
wants to merge 1 commit into from

Conversation

ErisDS
Copy link
Member

@ErisDS ErisDS commented Jun 28, 2017

This is a quick version for now, to ensure that enable is called and Ghost doesn't fail to start on reboot of servers.

I will try to revisit the better plan in #248 next week,

refs #216, #248

  • add an extra step to enable systemd on setup
    • this is what tells systemd to start the service on server restart
  • Update the template to contain docs link

refs TryGhost#216, TryGhost#248

- add an extra step to enable systemd on setup
   - this is what tells systemd to start the service on server restart
- Update the template to contain docs link
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 43.388% when pulling 72531f4 on ErisDS:systemd-enable into a82fba4 on TryGhost:master.

@acburdine
Copy link
Member

@ErisDS could I get you to rebase this against master? I won't be able to do it myself 😛

@acburdine
Copy link
Member

sooo...kind of stole the thunder on this as I wanted to get the full feature of enable/disable in. Would it be possible to update this PR/message just to update the docs link in the template?

@ErisDS
Copy link
Member Author

ErisDS commented Jul 3, 2017

Doesn't really matter how it gets in :) You are definitely more well-placed to do a rebase on this than I - I just tried and I don't know where any files are anymore and there are no instructions or explanations that I can find, so I'm going to have to relearn the codebase:

It's totally possible for you to rebase as PRs can be pushed to. If the only thing left to do is a text change then might as well just commit that directly?

@ErisDS ErisDS closed this Jul 3, 2017
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