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

Update README #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README #1

wants to merge 1 commit into from

Conversation

aviav
Copy link

@aviav aviav commented Jan 17, 2017

No description provided.

@@ -3,11 +3,12 @@
[![Gem Version](https://badge.fury.io/rb/ey_secrets.svg)](http://badge.fury.io/rb/ey_secrets)
[![Build Status](https://travis-ci.org/tf/ey_secrets.svg?branch=master)](https://travis-ci.org/tf/ey_secrets)

Manage sensible configuration files accross Engine Yard instances.
Manage sensitive configuration files across Engine Yard instances
within a dedicated config app.
Copy link
Owner

Choose a reason for hiding this comment

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

Do you mean "dedicated config repository"?

`[email protected]:companyrepo/software_config.git`, ey_secrets
expects to find the app repo at
`[email protected]:companyrepo/software.git`. You can instead provide
the app's name on Engine Yard using the `-a` option.
Copy link
Owner

Choose a reason for hiding this comment

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

Let's not present the naming scheme as a requirement. Instead let's just have the example be explicit and use -a. Then we can mention the naming scheme in a separate remark afterwards, as a way to skip the -a option.

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.

2 participants