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.

Sign up for GitHub

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

Jump to bottom

Documentation issue: topics/development/conventions/formulas.html #14900

Closed
tligda opened this issue Aug 11, 2014 · 3 comments · Fixed by #14921
Closed

Documentation issue: topics/development/conventions/formulas.html #14900

tligda opened this issue Aug 11, 2014 · 3 comments · Fixed by #14921
Labels
Documentation Relates to Salt documentation Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged
Milestone
Approved

Comments

@tligda
Copy link
Contributor

tligda commented Aug 11, 2014

Under "Adding a Formula directory manually", #2 shows the following:

.. code-block:: yaml

    file_roots:
      - /srv/salt
      - /srv/formulas/apache-formula

It should be the following:

.. code-block:: yaml

    file_roots:
      base:
        - /srv/salt
        - /srv/formulas/apache-formula
@tligda
Copy link
Contributor Author

tligda commented Aug 11, 2014

I'm happy to fork it, fix it and submit it. Just let me know.

@cachedout
Copy link
Contributor

We always welcome pull-requests. :] Go ahead and send us the correction and we'll get it included. Thanks much!

@cachedout cachedout added this to the Approved milestone Aug 11, 2014
@tligda
Copy link
Contributor Author

tligda commented Aug 12, 2014

Good fix! Let 'er rip!

nmadhok added a commit to nmadhok/salt that referenced this issue Aug 12, 2014
@nmadhok
Update formulas.rst
f99dac4 
Fixes saltstack#14900
@nmadhok nmadhok mentioned this issue Aug 12, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation Relates to Salt documentation Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged
Projects
None yet
Milestone
Approved
Development

Successfully merging a pull request may close this issue.

Update formulas.rst nmadhok/salt
2 participants
@cachedout @tligda