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

chore: fix markdownlint issues #3252

Merged
merged 2 commits into from
Sep 14, 2022

Conversation

pichlermarc
Copy link
Member

Which problem is this PR solving?

Running markdownlint revealed some issues with style and broken links. This PR addresses them (no fixes to the API markdown files for now). 🙂

@pichlermarc pichlermarc requested a review from a team September 14, 2022 06:40
@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Merging #3252 (a669cca) into main (9c102b7) will decrease coverage by 0.01%.
The diff coverage is n/a.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3252      +/-   ##
==========================================
- Coverage   93.30%   93.29%   -0.02%     
==========================================
  Files         203      203              
  Lines        6606     6606              
  Branches     1389     1389              
==========================================
- Hits         6164     6163       -1     
- Misses        442      443       +1     
Impacted Files Coverage Δ
...-trace-base/src/platform/node/RandomIdGenerator.ts 87.50% <0.00%> (-6.25%) ⬇️

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@dyladan dyladan merged commit ec56837 into open-telemetry:main Sep 14, 2022
@dyladan dyladan deleted the chore/fix-markdown branch September 14, 2022 15:21
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.

None yet

3 participants