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

Fix docs deployment #3044

Merged
merged 5 commits into from
Aug 6, 2022
Merged

Fix docs deployment #3044

merged 5 commits into from
Aug 6, 2022

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Jun 16, 2022

Docs deployment has failed for a while. We were also missing some of the packages now that we are back to a single monorepo.

@dyladan dyladan requested a review from a team June 16, 2022 21:00
@codecov
Copy link

codecov bot commented Jun 16, 2022

Codecov Report

Merging #3044 (9b3fe1f) into main (ddd0c6b) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3044      +/-   ##
==========================================
- Coverage   93.13%   93.11%   -0.02%     
==========================================
  Files         196      196              
  Lines        6406     6406              
  Branches     1350     1350              
==========================================
- Hits         5966     5965       -1     
- Misses        440      441       +1     
Impacted Files Coverage Δ
...-trace-base/src/platform/node/RandomIdGenerator.ts 87.50% <0.00%> (-6.25%) ⬇️

dyladan added 3 commits August 5, 2022 10:56
Testing shows that the docs generation uses about 3.9GB so a bit of
headroom seems like a good idea
Copy link
Member

@pichlermarc pichlermarc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks! 🙂

@legendecas legendecas merged commit 0c13343 into open-telemetry:main Aug 6, 2022
@dyladan dyladan deleted the docs-deploy branch August 8, 2022 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants