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

Increase coverage #1216

Merged
merged 6 commits into from
Feb 14, 2024
Merged

Increase coverage #1216

merged 6 commits into from
Feb 14, 2024

Conversation

renatomello
Copy link
Contributor

@renatomello renatomello commented Feb 13, 2024 •
edited
Loading

Checklist:

  • Reviewers confirm new code works as expected.
  • Tests are passing.
  • Coverage does not decrease.
  • Documentation is updated.

@renatomello renatomello added this to the Qibo 0.2.5 milestone Feb 13, 2024
@renatomello renatomello self-assigned this Feb 13, 2024
@renatomello renatomello linked an issue Feb 13, 2024 that may be closed by this pull request
Increase coverage #1215
Closed
@codecov Codecov
Copy link

codecov bot commented Feb 13, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (fb55eee) 99.96% compared to head (f4ca928) 100.00%.

Additional details and impacted files 
@@             Coverage Diff             @@
##           master     #1216      +/-   ##
===========================================
+ Coverage   99.96%   100.00%   +0.03%     
===========================================
  Files          70        70              
  Lines       10376     10376              
===========================================
+ Hits        10372     10376       +4     
+ Misses          4         0       -4
Flag Coverage Δ
unittests 100.00% <ø> (+0.03%) ⬆️

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.

@Simone-Bordoni Simone-Bordoni self-requested a review February 13, 2024 11:40
@renatomello
Copy link
Contributor Author

@scarrazza should be fixed now

@renatomello renatomello removed the request for review from BrunoLiegiBastonLiegi February 14, 2024 05:25
@renatomello renatomello added the quantum_info module PRs and issues related to the quantum information module label Feb 14, 2024
scarrazza
scarrazza approved these changes Feb 14, 2024
View reviewed changes
Copy link
Member

@scarrazza scarrazza left a comment

Choose a reason for hiding this comment

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

Thanks.

@scarrazza scarrazza added this pull request to the merge queue Feb 14, 2024
Merged via the queue into master with commit 4b4e6c3 Feb 14, 2024
23 checks passed
@renatomello renatomello deleted the coverage branch February 14, 2024 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Simone-Bordoni Simone-Bordoni Simone-Bordoni approved these changes

@scarrazza scarrazza scarrazza approved these changes

@BrunoLiegiBastonLiegi BrunoLiegiBastonLiegi Awaiting requested review from BrunoLiegiBastonLiegi

Assignees

@renatomello renatomello

Labels
quantum_info module PRs and issues related to the quantum information module
Projects
None yet
Milestone
Qibo 0.2.5
Development

Successfully merging this pull request may close these issues.

Increase coverage
3 participants
@renatomello @scarrazza @Simone-Bordoni