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 consuming commons from the Ruby CNB main branch #725

Merged
merged 2 commits into from
Nov 17, 2023

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Nov 17, 2023

Since the feature branch no longer exists.

This will fix Dependabot erroring out with:

Errored with the message "Dependabot couldn't fetch the branch/reference for commons"

This also required adding fun_run as an explicit dependency, due to:
heroku/buildpacks-ruby#232

GUS-W-14513823.

Since the feature branch no longer exists.

This will fix Dependabot erroring out with:

```
Errored with the message "Dependabot couldn't fetch the branch/reference for commons"
```
@edmorley edmorley self-assigned this Nov 17, 2023
@edmorley edmorley marked this pull request as ready for review November 17, 2023 17:10
@edmorley edmorley requested a review from a team as a code owner November 17, 2023 17:10
Fixes:

```
error[E0432]: unresolved import `commons::fun_run`
 --> buildpacks/nodejs-npm-install/src/errors.rs:3:14
  |
3 | use commons::fun_run::CmdError;
  |              ^^^^^^^ could not find `fun_run` in `commons`
```
etc

Which is due to:
heroku/buildpacks-ruby#232
@edmorley edmorley enabled auto-merge (squash) November 17, 2023 17:19
@edmorley edmorley merged commit f1bbcea into main Nov 17, 2023
21 checks passed
@edmorley edmorley deleted the edmorley/commons-main branch November 17, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants