-
Notifications
You must be signed in to change notification settings - Fork 104
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
doc: explanation of how libraries depend on one another #3469
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3469 +/- ##
==========================================
- Coverage 77.35% 77.35% -0.01%
==========================================
Files 1076 1076
Lines 68922 68922
==========================================
- Hits 53317 53315 -2
- Misses 15605 15607 +2 ☔ View full report in Codecov by Sentry. |
2f18e0d
to
e098fd0
Compare
e098fd0
to
71a9348
Compare
doc/sphinx/explanation/libraries.md
Outdated
Mir builds a number of static libraries that are linked to internally by | ||
`mirserver`. These libraries are: |
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.
Your list includes some of the "object" libraries. I think there's only one static library linked by mirserver (kms_utils).
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.
Great start!
Personally, I'd drop the section about object libraries as it would need more work I don't think there's enough benefit to justify that
doc/sphinx/explanation/libraries.md
Outdated
- `miral` and `miroil` are intended to be the primary user-facing libraries as | ||
nothing else depends on them |
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.
It is miral
and mircore
that are intended to be the primary user-facing libraries.
Miroil needs special mention as a "compatibility layer" for Lomiri that is not intended for general consumption
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.
LGTM
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.
Typo aside, looks good to me.
Check out https://canonical-mir--3469.com.readthedocs.build/3469/explanation/libraries/