-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat: Make various data publicly accessible #875
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #875 +/- ##
==========================================
- Coverage 85.59% 85.48% -0.12%
==========================================
Files 78 78
Lines 14432 14450 +18
Branches 14432 14450 +18
==========================================
- Hits 12353 12352 -1
- Misses 1442 1460 +18
- Partials 637 638 +1 ☔ View full report in Codecov by Sentry. |
df0a3f3
to
815a8c8
Compare
@@ -246,6 +251,12 @@ impl TypeArg { | |||
} | |||
} | |||
|
|||
impl TypeArgVariable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no docstring here, why isn't CI catching this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uh, seems I missed setting lints.workspace = true
in the package's Cargo.toml when I created the workspace...
I can fix it in a separate PR, or you can add
[lints]
workspace = true
in quantinuum-hugr/Cargo.toml
.
Good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #876
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Just add the missing doc.
815a8c8
to
f6e3d7f
Compare
No description provided.