-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
lcm_ #21626
lcm_ #21626
Conversation
Thanks for contributing to Ivy! 😊👏 |
Thank you for your contribution to Ivy! Can you please resolve the merge conflicts so that your PR can be merged? |
53860c3
to
a5c3028
Compare
Hi @theRealBird, |
@NiteshK84 Thanks for resolving the conflicts Nitesh. For some reason, your PR is introducing a lot of new failures (unrelated to your changes). Seems like there is a bug with the CI testing but could you please take a look once too? Thanks! |
Hey @theRealBird, I have fixed the conflicts, but I have not fixed these since they didn't belong to me, but for sure they will cause the CI lint test to fail, please tell me weather should i fix theme or not? |
59ade45
to
c02cc5e
Compare
|
added func and test
Hey @theRealBird, i fixed all the failures, |
Closes #21625