Skip to content

Conversation

@Pearl1594
Copy link
Contributor

@codecov
Copy link

codecov bot commented Dec 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.06%. Comparing base (54bc150) to head (884980b).

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #10125   +/-   ##
=========================================
  Coverage     16.06%   16.06%           
- Complexity    12863    12864    +1     
=========================================
  Files          5642     5642           
  Lines        493856   493856           
  Branches      59860    59860           
=========================================
+ Hits          79324    79326    +2     
  Misses       405749   405749           
+ Partials       8783     8781    -2     
Flag Coverage Δ
uitests 4.02% <ø> (ø)
unittests 16.90% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@resmo
Copy link
Member

resmo commented Dec 18, 2024

To be honest, I don't really like this PR. Not that it matters to me whether it's Christmas or any other festival in any religion. It shouldn't have anything to do with this project... and a simple

Merry Christmas to the ones who celebrate

on the ML should be enough.

Just my 2 cents.

@DaanHoogland
Copy link
Contributor

To be honest, I don't really like this PR. Not that it matters to me whether it's Christmas or any other festival in any religion. It shouldn't have anything to do with this project... and a simple

Merry Christmas to the ones who celebrate

on the ML should be enough.

Just my 2 cents.

I like your killjoy attitude @resmo (no sarcasm, really)

@gsirett
Copy link

gsirett commented Dec 18, 2024

I'm normally accused of being the Christmas grinch, I think @resmo is bidding for my title :-)

Happy Christmas (to those that celebrate) and a peaceful new year to everybody in this community

@sonarqubecloud
Copy link

Copy link
Contributor

@nvazquez nvazquez left a comment

Choose a reason for hiding this comment

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

Merry Christmas and happy new year all! 💪🏼

@andrijapanicsb
Copy link
Contributor

@resmo pls improve on the code so that we can eventually merge it. I see sooooooo many people are asked to review - this is hardly ever going to be merged, damn it.... (not that I care about holidays that much :D )

Copy link
Contributor

@andrijapanicsb andrijapanicsb left a comment

Choose a reason for hiding this comment

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

What Christmas, 25th December 2024 or 7th January 2025?
Code is unclear. There are no proper logging messages. Have the smoke tests passed for major hypervisors?

Copy link
Member

@rohityadavcloud rohityadavcloud left a comment

Choose a reason for hiding this comment

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

Happy Christmas and holidays.

Copy link
Member

@resmo resmo left a comment

Choose a reason for hiding this comment

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

🤷

@abh1sar
Copy link
Collaborator

abh1sar commented Dec 19, 2024

Merry Christmas and Happy Holidays 🎉 !

Copy link
Contributor

@JoaoJandre JoaoJandre left a comment

Choose a reason for hiding this comment

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

Happy holidays to everyone 🎅

@msinhore
Copy link

Merry Christmas and Happy New Year everybody!

Copy link
Contributor

@BryanMLima BryanMLima left a comment

Choose a reason for hiding this comment

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

Happy holidays everyone!!

Copy link
Collaborator

@hsato03 hsato03 left a comment

Choose a reason for hiding this comment

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

Happy holidays! 🎊

@DaanHoogland
Copy link
Contributor

What Christmas, 25th December 2024 or 7th January 2025? Code is unclear. There are no proper logging messages. Have the smoke tests passed for major hypervisors?

@andrijapanicsb , since you are blocking this High-prio and time-sensitive PR, please add suggestions to make its progress possible. i.e. add log messages to add/specify the missing smoke tests.

Please make it as easy on the contributor as possible to have their changes adopted!

Copy link
Contributor

@sureshanaparti sureshanaparti left a comment

Choose a reason for hiding this comment

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

Happy Christmas and New Year!

Copy link
Contributor

@vladimirpetrov vladimirpetrov left a comment

Choose a reason for hiding this comment

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

Happy holidays to all of you!

Copy link
Contributor

@kiranchavala kiranchavala left a comment

Choose a reason for hiding this comment

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

Happy Christmas and New Year!

Copy link
Contributor

@harikrishna-patnala harikrishna-patnala left a comment

Choose a reason for hiding this comment

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

Merry Christmas and happy new year all! Looking forward to another great year of collaboration!

Copy link
Member

@soreana soreana left a comment

Choose a reason for hiding this comment

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

Wishing everyone a Merry Christmas and a Happy New Year! Excited for another fantastic year of working together!

@rohityadavcloud rohityadavcloud merged commit 631bba2 into main Dec 24, 2024
48 of 49 checks passed
@rohityadavcloud rohityadavcloud deleted the 2024-holidays branch December 24, 2024 11:32
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.