Skip to content

Add usage information line to readme#22

Merged
hmac merged 1 commit intomasterfrom
add-usage-info
Nov 30, 2017
Merged

Add usage information line to readme#22
hmac merged 1 commit intomasterfrom
add-usage-info

Conversation

@nickcampbell18
Copy link
Copy Markdown
Contributor

@nickcampbell18 nickcampbell18 commented Nov 30, 2017

Comment thread README.md Outdated

Draupnir is a tool that provides on-demand Postgres databases with preloaded data.

**Looking to use Draupnir in your development environment? The instructions are [in the developer handbook](https://developer-handbook.app.gocardless.io/chapters/your_first_week/tools.html#draupnir).**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure about this, as I thought our intentions were to open-source draupnir. I'm still very keen that we do so, which makes including the content here a bit strange?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Feels like we've just made this PR extremely confusing should we ever open source it!

Comment thread README.md Outdated

Draupnir is a tool that provides on-demand Postgres databases with preloaded data.

**Looking to use Draupnir? Please check in the developer handbook**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

check the?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No strong feels!

@hmac hmac merged commit 8019d06 into master Nov 30, 2017
@hmac hmac deleted the add-usage-info branch November 30, 2017 13:46
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