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

Fix Master build failure #13302

Merged

Conversation

jepenven-silabs
Copy link
Contributor

Problem

#13071 seems to have broken a couple of CI job

Change overview

Fix the build failure

Testing

EFR32 platform now compiles

@jepenven-silabs jepenven-silabs changed the title Fix build failure Fix Master build failure Jan 3, 2022
Copy link
Collaborator

@electrocucaracha electrocucaracha left a comment

Choose a reason for hiding this comment

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

/lgtm

This case is interesting the CI didn't report a failure in its PR but the error was raised in master branch

@jepenven-silabs jepenven-silabs merged commit 462dd77 into project-chip:master Jan 3, 2022
@bzbarsky-apple
Copy link
Contributor

This case is interesting the CI didn't report a failure in its PR but the error was raised in master branch

That's because it was a merge conflict, no? @electrocucaracha

@electrocucaracha
Copy link
Collaborator

This case is interesting the CI didn't report a failure in its PR but the error was raised in master branch

That's because it was a merge conflict, no? @electrocucaracha

It could be or at least it's a possibility

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.

5 participants