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.

Sign up for GitHub

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

Jump to bottom

Check metadata if err is nil #2312

Merged
merged 1 commit into from
Mar 4, 2022
Merged

Check metadata if err is nil #2312

merged 1 commit into from
Mar 4, 2022

Conversation

teresaromero
Copy link
Member

Signed-off-by: Teresa Romero teresa@okteto.com

Proposed changes

  • Only try to get digest if build has gone with err nil

@teresaromero
refactor: check metadata if err is nil
cc5f6c5 
Signed-off-by: Teresa Romero <teresa@okteto.com>
@teresaromero teresaromero requested a review from a team March 3, 2022 15:45
@teresaromero teresaromero merged commit 40f7d8f into master Mar 4, 2022
@teresaromero teresaromero deleted the tere/fix-check-metadata-build branch March 4, 2022 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ifbyol ifbyol ifbyol approved these changes

@jLopezbarb jLopezbarb jLopezbarb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@teresaromero @ifbyol @jLopezbarb