Skip to content

Conversation

@Jcambass
Copy link
Contributor

@Jcambass Jcambass commented Jul 3, 2019

Heroku recommends to always specify a specific release when using a 3rd party Buildpack.

Many people don't look at the heroku docs but only follow the README instructions of the repo. This causes some (sometimes hard to debug) problems because a unchanged build can suddenly fail because of changes on the Buildpacks master branch.
To prevent people from accidentally using the master branch on the Buildpack in production I've included instructions on how to specify a specific Buildpack release in the README.

Related issue #115

@jyee
Copy link
Contributor

jyee commented Jul 5, 2019

@Jcambass Thanks, this is a good update. If I can make one request, let's keep the language a bit more formal and change "wanna" to "want to".

The README is automatically pulled into a few different locations (e.g. Heroku's buildpack directory and Datadog's docs) where automatic language translation is applied. Using more formal language helps with automated translation.

Heroku recommends to always specify a specific release when [using a 3rd party Buildpack](https://devcenter.heroku.com/articles/buildpacks#using-a-third-party-buildpack).

Many people don't look at the heroku docs but only follow the README instructions of the repo. This causes some (sometimes hard to debug) problems because a unchanged build can suddenly fail because of changes on the Buildpacks master branch.
To prevent people from accidentally using the master branch on the Buildpack in production I've included instructions on how to specify a specific Buildpack release in the README.

Related issue #115
@Jcambass
Copy link
Contributor Author

Jcambass commented Jul 5, 2019

@jyee Thanks for hitting me at this! I've adjusted the wording.

@jyee
Copy link
Contributor

jyee commented Jul 8, 2019

@Jcambass thanks!

@jyee jyee merged commit 81d29ee into DataDog:master Jul 8, 2019
@Jcambass Jcambass deleted the change_readme_example_to_specify_exact_version branch July 8, 2019 11:45
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.

2 participants