-
Notifications
You must be signed in to change notification settings - Fork 210
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
chore: add Readme.md links in each Cargo.toml #4030
Conversation
Added Readme.md links in those Crate.toml where there exists a README.md file in that individual crate
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4030 +/- ##
=======================================
Coverage 57.73% 57.73%
=======================================
Files 193 193
Lines 42948 42948
=======================================
+ Hits 24794 24798 +4
+ Misses 18154 18150 -4 ☔ View full report in Codecov by Sentry. |
Cargo.toml
Outdated
@@ -61,6 +61,7 @@ keywords = ["bitcoin", "lightning", "chaumian", "e-cash", "federated"] | |||
threshold_crypto = { version = "0.1", package = "fedimint-threshold-crypto" } | |||
tonic_lnd = { version = "0.1.3", package="fedimint-tonic-lnd", features = ["lightningrpc", "routerrpc"] } | |||
cln-rpc = { package = "fedimint-cln-rpc", version = "0.4.0" } | |||
fedimint-aead = { version = "0.0.1" } |
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 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.
By Mistake, I thought that I have to include this line in root Cargo.toml file as per the screenshot :)
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.
Doesn't fully solve #3924 since crates without README still don't have any linked, but it's an improvement.
For the crates not having their own README -> I have to give link of the root Readme file? |
I think that's a complicated problem since the root README isn't included when publishing sub-crates by default (see my comment in the issue). So this requires some Cargo wizardry, which is probably not worth it rn. |
Okk, |
@elsirion |
Interesting, then let's try this in another PR! |
Removed the relative paths of root README.md file from the unpublished crates.(from fedimint-load-test-tool crate also -> It is unpublished crate but was merged fedimint#4030)
Added Readme.md links in those Crate.toml where there exists a README.md file in that individual crate
#3924