Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

feature: better documentation for config files & locations #591

Merged
merged 1 commit into from
Aug 10, 2016
Merged

Conversation

jrconlin
Copy link
Member

@jrconlin jrconlin commented Aug 9, 2016

Add documentation about the value of the configuration files and
where they can be located on the system.

Closes #572

@pjenvey r?

@pjenvey
Copy link
Member

pjenvey commented Aug 9, 2016

Did you git mv the configs? I don't see the .sample files here (they weren't staged?)


* in the $HOME or current directory (prefixed by a period '.')

* in the configs subdirectory
Copy link
Member

Choose a reason for hiding this comment

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

Mention configs before $HOME/current since that's the order they're checked

@pjenvey
Copy link
Member

pjenvey commented Aug 9, 2016

Also there's a lingering reference to configs/autopush_shared.ini in running.rst

@jrconlin
Copy link
Member Author

jrconlin commented Aug 9, 2016

Weird, I did git mv the configs. Might have gotten stomped on the rebase.

Add documentation about the value of the configuration files and
where they can be located on the system.

Closes #572
@codecov-io
Copy link

Current coverage is 100% (diff: 100%)

Merging #591 into master will not change coverage

@@           master   #591   diff @@
====================================
  Files          42     42          
  Lines        8798   8798          
  Methods         0      0          
  Messages        0      0          
  Branches        0      0          
====================================
  Hits         8798   8798          
  Misses          0      0          
  Partials        0      0          

Powered by Codecov. Last update f237070...7186982

@pjenvey
Copy link
Member

pjenvey commented Aug 10, 2016

r+

@pjenvey pjenvey merged commit 8415600 into master Aug 10, 2016
@pjenvey pjenvey deleted the feat/572 branch August 10, 2016 00:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants