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

Docsy update to 5b49200 and remove tmp layout override #1479

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jun 24, 2022

Generate site-file diff:

$ npm run check-links
...
$ find public/ -name "*.html" -exec perl -pi -e 's/ aria-disabled="true" class="btn-link disabled"//g;' {} \;   
$ npm run cd:public -- git diff -bw --ignore-blank-lines -I "<html" -I "fa-(github|tasks) fa-fw" -I "aria-controls=\"td-(docs|section)-nav\"" -- . ':(exclude)*.[^h]*'
...
diff --git a/community/index.html b/community/index.html
index 5cbe4c54..665584e9 100644
--- a/community/index.html
+++ b/community/index.html
@@ -227,7 +227,7 @@ improve our workflow and …">
 </ul>
 
         
-        <p>You can find out how to contribute to these docs in our <a href="../docs/contribution-guidelines/">Contribution Guidelines</a>
+        <p>You can find out how to contribute to OpenTelemetry in our <a href="/docs/contribution-guidelines/">Contribution Guidelines</a>
     </div>
 </section>

The find-perl command strips out extra attributes that were added to breadcrumbs via google/docsy#1068.

@chalin chalin added the CI/infra CI & infrastructure label Jun 24, 2022
@chalin chalin requested a review from a team as a code owner June 24, 2022 12:17
@chalin chalin merged commit 044335a into open-telemetry:main Jun 24, 2022
@chalin chalin deleted the chalin-im-docsy-update-2022-06-24 branch June 24, 2022 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/infra CI & infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant