Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Start parity with systemd #1967

Merged
merged 7 commits into from
Aug 30, 2016
Merged

Start parity with systemd #1967

merged 7 commits into from
Aug 30, 2016

Conversation

gabx
Copy link
Contributor

@gabx gabx commented Aug 19, 2016

Systemd is now the default way to start services in mots Linux distro. Use the provided parity.service file to start parity as regular user (best practice than starting the service as root).
Readme has been updated acordingly

@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Ethcore CLA Bot

@gabx gabx mentioned this pull request Aug 19, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 86.934% when pulling 87b882d on gabx:master into 34de330 on ethcore:master.

@gavofyork gavofyork added the A0-pleasereview 🤓 Pull request needs code review. label Aug 19, 2016
@@ -84,9 +84,21 @@ $ cargo build --release

This will produce an executable in the `./target/release` subdirectory.

To get started, just run
## Start parity
### manually
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capitalise Manually

@gavofyork gavofyork added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. and removed A0-pleasereview 🤓 Pull request needs code review. labels Aug 21, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 86.924% when pulling 32530c6 on gabx:master into 34de330 on ethcore:master.

@gavofyork
Copy link
Contributor

still missing some capitalisations.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 86.861% when pulling ef0bb69 on gabx:master into 34de330 on ethcore:master.

@@ -86,19 +86,19 @@ This will produce an executable in the `./target/release` subdirectory.

## Start parity
### manually
To start manually parity, just run
To start Manually Parity, just run
Copy link
Contributor

@rphmeier rphmeier Aug 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this line should be "Parity manually", but the line above it should say "Manually"

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 86.898% when pulling a8d2647 on gabx:master into 34de330 on ethcore:master.

@gabx
Copy link
Contributor Author

gabx commented Aug 26, 2016

Is there anything still wrong ? Did I forget a capitalization ?

Updating wording a bit and fixed remaining capitalization issues.
@arkpar
Copy link
Collaborator

arkpar commented Aug 30, 2016

Made a PR to fix remaining issues here: https://github.com/gabx/parity/pull/1

@rphmeier rphmeier added A8-looksgood 🦄 Pull request is reviewed well. and removed A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. labels Aug 30, 2016
@rphmeier
Copy link
Contributor

Probably doesn't need CI.

@rphmeier rphmeier merged commit 6945dc3 into openethereum:master Aug 30, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 86.373% when pulling 0e0e770 on gabx:master into 34de330 on ethcore:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants