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

feat(base): generate new image with version 18.6.0 #715

Conversation

ivangabriele
Copy link

@ivangabriele ivangabriele commented Jul 26, 2022

Splitting #712

@ivangabriele
Copy link
Author

@admah Don't hesitate to tell me if anything is missing. I took the liberty to clean some docs.

admah
admah previously approved these changes Jul 29, 2022
Copy link
Contributor

@admah admah left a comment

Choose a reason for hiding this comment

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

LGTM.

@admah
Copy link
Contributor

admah commented Jul 29, 2022

@ivangabriele it looks like there are some failures in testing unrelated to this PR, but that we will need to get sorted out before this is merged. Just wanted to give you a heads up.

@ivangabriele
Copy link
Author

ivangabriele commented Aug 1, 2022

@admah Thanks for the heads up ^^. I'll rebase this branch once master is ready.

@admah
Copy link
Contributor

admah commented Aug 4, 2022

@ivangabriele master is good to go. Update this branch and I'll take a look. Thanks!

@ivangabriele ivangabriele force-pushed the ivan/feat-base-add-new-image-for-node-18-6-0 branch from 98ab89a to b1dacc0 Compare August 21, 2022 18:18
@ivangabriele ivangabriele force-pushed the ivan/feat-base-add-new-image-for-node-18-6-0 branch from 6fd639e to 1b84de7 Compare August 21, 2022 18:41
@ivangabriele
Copy link
Author

@admah Sorry for the late update! I just rebased ;). I'm not sure why the CI don't run the new generated job, I guess it's because of Github branch protection rules ^^.

@lmiller1990
Copy link
Contributor

I made a new PR since it's easier than resolving conflicts: #738

@lmiller1990 lmiller1990 mentioned this pull request Sep 1, 2022
@admah
Copy link
Contributor

admah commented Sep 1, 2022

Closing this now that #738 has been merged.

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.

None yet

3 participants