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

Update README.md #278

Merged
merged 2 commits into from
Apr 23, 2018
Merged

Update README.md #278

merged 2 commits into from
Apr 23, 2018

Conversation

spider1163
Copy link
Contributor

cf. #214

@nemchik
Copy link
Collaborator

nemchik commented Apr 16, 2018

This is a good start, but the ATK folder might not always be in /opt/. Mine for example is in ~/.

@spider1163
Copy link
Contributor Author

It is based on /opt folder you set in "Installation and Usage" section of README.
I assumed user who changed the git clone folder would understand the nuance.
I can however make it more general. What do you think of : [install-folder]/AtoMiC-ToolKit/backups ?

@nemchik
Copy link
Collaborator

nemchik commented Apr 16, 2018

The /opt/ folder was introduced as the default folder some time after I had initially installed ATK. Maybe something as simple as saying The default location for stored backups is /opt/AtoMiC-ToolKit/backups and as you said people who have installed it elsewhere probably know what to look for.

@nemchik
Copy link
Collaborator

nemchik commented Apr 21, 2018

Looks good to me.

@TommyE123 TommyE123 changed the base branch from master to develop April 23, 2018 18:39
@TommyE123 TommyE123 merged commit 4d994ec into anandslab:develop Apr 23, 2018
@TommyE123
Copy link
Collaborator

@spider1163 Thanks ever so much for your contribution, Its very much appreciated.
Any further improvements are more than welcome :-)

@nemchik and I are both on Gitter if you wanted to discuss any ideas.

Cheers
Tom

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