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

add different scenarios to the Quickstart section #1205

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

sdondley
Copy link
Contributor

No description provided.

Copy link
Owner

@lra lra left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for this

README.md Outdated
@@ -597,4 +654,4 @@ remember, and it corresponds with the whole idea of Mackup: the simpler – the

## Where can I find more information

In the [doc](doc) directory.
I
Copy link
Owner

Choose a reason for hiding this comment

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

Typo?

README.md Outdated
```

Save the file and then run the same commands given in Scenario 1
above to install andsync your apps. Done!
Copy link
Owner

Choose a reason for hiding this comment

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

andsync

README.md Outdated

As you get more comfortable with Mackup and want to add new
applications to sync, simply add the name of the supported application
to the `.mackup.cfg` to the end of the file on a new line by itself.
Copy link
Owner

Choose a reason for hiding this comment

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

Can you specify how? (mackup list)

Copy link
Contributor Author

@sdondley sdondley Sep 2, 2018

Choose a reason for hiding this comment

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

Can you please clarify this? I'm not sure what you are looking for here. Other changes were made.

Copy link
Owner

Choose a reason for hiding this comment

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

Just give a hint for your readers to know how they can know what to add there, telling them to run mackup list and to copy past an app name from the list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh OK. Sure, will take care of this when I get a chance.

README.md Outdated
the configurations files on the different machines they work on and are
ready for Mackup to end their misery.

**Scenario 1:** You want to push **all** configuration settings from **all**
Copy link
Owner

Choose a reason for hiding this comment

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

Can you use headers to title each scenario?
e.g. ### Scenarion 2

README.md Outdated
@@ -21,6 +21,19 @@ Keep your application settings in sync.

## Quickstart

Instructions for three common use cases are below.

- **Scenario 1** is the quickest and easiest but gives you no control over
Copy link
Owner

Choose a reason for hiding this comment

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

Would be cool to link to each scenario

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