-
Notifications
You must be signed in to change notification settings - Fork 729
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
Use Google Analytics 4 ID for v1.7 #2962
Conversation
Signed-off-by: Patrice Chalin <[email protected]>
Signed-off-by: Patrice Chalin <[email protected]>
For more information visit the [Dapr branch structure](https://docs.dapr.io/contributing/contributing-docs/#branch-guidance) document. | ||
For more information visit the [Dapr branch structure](https://docs.dapr.io/contributing/docs-contrib/contributing-docs/#branch-guidance) document. |
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.
Update necessary so that the "validate links" GH action passes.
By the way, when website pages are moved, you should consider adding redirect rules to that users don't get 404s.
Signed-off-by: Patrice Chalin <[email protected]>
Well, I fixed the first few broken external links, but I'm going to stop now. Hopefully this PR can still be merged, or someone else can track down and fix all of the broken external links. |
Signed-off-by: Patrice Chalin <[email protected]>
@@ -36,5 +36,4 @@ jobs: | |||
pip3 install setuptools wheel twine tox mechanical-markdown | |||
- name: Check Markdown Files | |||
run: | | |||
for name in `find . -name "*.md"`; do echo -e "------\n$name" ; mm.py -l $name || exit 1 ;done | |||
|
|||
for name in `find . -name "*.md"`; do echo -e "------\n$name" ; mm.py -l $name ;done |
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.
Since these are legacy docs, and hence probably no longer actively updated, I figured that it would be ok to allow link validation to fail softly and not break the build.
Signed-off-by: Patrice Chalin <[email protected]>
Ok, the build is passing now. PTAL @greenie-msft et al. |
Stale PR, paging all reviewers |
/cc @msfussell @greenie-msft