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

📝 Rework existing project documentation #789

Merged
merged 7 commits into from
Jan 3, 2025
Merged

Conversation

burgholzer
Copy link
Member

Description

This PR reworks most of the existing project documentation. Some of this was triggered by the ongoing JOSS submission, some was long overdue.
While extending the documentation, a small bug in the OpenQASM dump of classic controlled operations has been found and subsequently fixed.

While this PR mostly touches existing documentation, a follow-up PR will add documentation for the DD and the ZX package.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
…utput as part of the documentation

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer added documentation Improvements or additions to documentation Core Anything related to the Core library and IR labels Jan 3, 2025
@burgholzer burgholzer self-assigned this Jan 3, 2025
Copy link

codecov bot commented Jan 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.2%. Comparing base (653c606) to head (5ca5391).
Report is 8 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #789   +/-   ##
=====================================
  Coverage   92.2%   92.2%           
=====================================
  Files        125     125           
  Lines      13759   13764    +5     
  Branches    2165    2167    +2     
=====================================
+ Hits       12696   12701    +5     
  Misses      1063    1063           
Flag Coverage Δ
cpp 92.0% <100.0%> (+<0.1%) ⬆️
python 99.7% <ø> (ø)
Files with missing lines Coverage Δ
...-core/ir/operations/ClassicControlledOperation.hpp 96.9% <100.0%> (+0.2%) ⬆️

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer enabled auto-merge January 3, 2025 19:37
@burgholzer burgholzer merged commit 449a5b1 into main Jan 3, 2025
30 checks passed
@burgholzer burgholzer deleted the documentation branch January 3, 2025 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Anything related to the Core library and IR documentation Improvements or additions to documentation
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant