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.

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

Switch to heroku/builder:22 in docs and tests #399

Merged
merged 1 commit into from
Jun 13, 2022
Merged

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Jun 10, 2022

Since it supersedes heroku/buildpacks:20:
https://github.com/heroku/builder#heroku-builder-images

Whilst updating the example build output in the README, I noticed that the specified commands were incorrect (eg wrong buildpack --path), so those have been fixed. In addition, I've switched the hello world example to use the wildcard stack, since the code is not stack-specific, and it will save us having to remember to update it in the future.

Lastly, updating the Ruby example buildpack to heroku/builder:22 revealed that the buildpack was actually broken and relying on system Ruby, so that buildpack has been excluded from using the new builder until that is resolved (#398).

Since it supersedes `heroku/buildpacks:20`:
https://github.com/heroku/builder#heroku-builder-images

Whilst updating the example build output in the README, I noticed
that the specified commands were incorrect (eg wrong buildpack path),
so those have been fixed.

In addition, I've switched the hello world example to use the wildcard
stack, since the code is not stack-specific, and it will save us having to
remember to update it in the future.

Lastly, updating the Ruby example buildpack to `heroku/builder:22`
revealed that the buildpack was actually broken and relying on
system Ruby, so that buildpack has been excluded from using the
new builder until that is resolved:
#398
@edmorley edmorley added documentation Improvements or additions to documentation skip changelog examples libcnb-test labels Jun 10, 2022
@edmorley edmorley self-assigned this Jun 10, 2022
@edmorley edmorley marked this pull request as ready for review June 10, 2022 21:50
@edmorley edmorley requested a review from a team as a code owner June 10, 2022 21:50
@edmorley edmorley merged commit b5d4d87 into main Jun 13, 2022
@edmorley edmorley deleted the edmorley/builder-22 branch June 13, 2022 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples libcnb-test skip changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants