Skip to content

Conversation

@colincasey
Copy link
Contributor

In v5, the following provides are no longer used:

  • node_build_scripts

This is replaced by the heroku/nodejs provide which supports the same buildplan config.

See heroku/buildpacks-nodejs#1169 for more context.

W-19524996

In v5, the following provides are no longer used:
- node_build_scripts

This is replaced by the `heroku/nodejs` provide which supports the same buildplan config.

See heroku/buildpacks-nodejs#1169 for more context.
@colincasey colincasey requested a review from mars September 3, 2025 19:59
@colincasey colincasey self-assigned this Sep 3, 2025
@colincasey colincasey marked this pull request as ready for review September 3, 2025 20:04
Copy link
Member

@mars mars left a comment

Choose a reason for hiding this comment

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

Thank you for this work simplifying the CNB heroku/nodejs experience! Looks good.

// component buildpacks responsible for installing dependencies for the detected
// package manager (i.e.; npm, pnpm, or Yarn). This needs to be disabled so that
// the build process can be deferred to the release-build phase.
enabled = false
Copy link
Member

Choose a reason for hiding this comment

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

Having this enabled configuration now belong to heroku/nodejs in general seems potentially confusing. Maybe this flag should be given a new name like build_scripts_enabled?

@mars mars merged commit 7311279 into main Sep 8, 2025
3 checks passed
@mars mars deleted the update_nodejs_requires_for_v5 branch September 8, 2025 20:47
heroku-linguist bot added a commit that referenced this pull request Sep 8, 2025
## heroku/static-web-server

- No changes.

## heroku/website

### Changed

- Updated `heroku/static-web-server` to `1.0.8`.
- Updated `heroku/website-public-html` to `1.0.8`.

## heroku/website-ember

### Changed

- Use `heroku/nodejs` buildplan entry for Node.js buildpack. ([#49](#49))

## heroku/website-nodejs

### Changed

- Updated `heroku/nodejs` to `5.0.0`. ([#49](#49))
- Updated `heroku/static-web-server` to `1.0.8`.
- Updated `heroku/website-ember` to `1.0.8`.

## heroku/website-public-html

- No changes.
@heroku-linguist heroku-linguist bot mentioned this pull request Sep 8, 2025
heroku-linguist bot added a commit that referenced this pull request Sep 8, 2025
## heroku/static-web-server

- No changes.

## heroku/website

### Changed

- Updated `heroku/static-web-server` to `1.0.8`.
- Updated `heroku/website-public-html` to `1.0.8`.

## heroku/website-ember

### Changed

- Use `heroku/nodejs` buildplan entry for Node.js buildpack. ([#49](#49))

## heroku/website-nodejs

### Changed

- Updated `heroku/nodejs` to `5.0.0`. ([#49](#49))
- Updated `heroku/static-web-server` to `1.0.8`.
- Updated `heroku/website-ember` to `1.0.8`.

## heroku/website-public-html

- No changes.

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
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