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

Consolidate size checks in container builds #837

Merged
merged 1 commit into from
Feb 23, 2023

Conversation

karlhigley
Copy link
Contributor

@karlhigley karlhigley commented Feb 23, 2023

We had two PRs yesterday (#832 and #836) that conflicted but weren't detected as a conflict. This blends the consequences of those PRs.

We had two PRs yesterday that conflicted but weren't detected as a conflict. This blends the consequences of those PRs.
@karlhigley karlhigley added chore Infrastructure update ci labels Feb 23, 2023
@karlhigley karlhigley added this to the Merlin 23.03 milestone Feb 23, 2023
@karlhigley karlhigley requested a review from jperez999 February 23, 2023 15:59
@karlhigley karlhigley self-assigned this Feb 23, 2023
@github-actions
Copy link

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-837

@jperez999 jperez999 merged commit 592939f into main Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Infrastructure update ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants