Skip to content

Conversation

@rajesh-nodle
Copy link
Contributor

R.Rajeshkumar added 2 commits May 20, 2022 11:12
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@codecov
Copy link

codecov bot commented May 20, 2022

Codecov Report

Merging #606 (f0594f6) into master (16547f6) will not change coverage.
The diff coverage is n/a.

❗ Current head f0594f6 differs from pull request most recent head 5deec70. Consider uploading reports for the commit 5deec70 to get more accurate results

@@           Coverage Diff           @@
##           master     #606   +/-   ##
=======================================
  Coverage   82.28%   82.28%           
=======================================
  Files          66       66           
  Lines        7992     7992           
=======================================
  Hits         6576     6576           
  Misses       1416     1416           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16547f6...5deec70. Read the comment docs.

@rajesh-nodle
Copy link
Contributor Author

rajesh-nodle commented May 20, 2022

Step Build the docs looks OK after eliminitaing the autodoc build on dependency, looks like issue is not on local workspace.

run: cargo doc --all --no-deps

Now its getting blocked at step Deploy the docs

May be good to check by increasing the storage of the git CI/CD runtime.

Leave it to @noisefloors , to take it forward ...

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@rajesh-nodle rajesh-nodle requested a review from a team May 20, 2022 09:23
ETeissonniere
ETeissonniere previously approved these changes May 20, 2022
@ETeissonniere ETeissonniere self-requested a review May 20, 2022 19:53
@ETeissonniere ETeissonniere dismissed their stale review May 20, 2022 19:54

CI autodoc was skipped, will let the team review this

@aliXsed aliXsed merged commit 9de488c into NodleCode:master May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants