Add Website Preview for New Documentation Page and Fixed Links#119
Add Website Preview for New Documentation Page and Fixed Links#119AshwinSriram11 merged 5 commits intoprometheus-operator:mainfrom Dharma-09:main
Conversation
AshwinSriram11
left a comment
There was a problem hiding this comment.
Nice 👍 ! Just a few more things to watch for.
The tests are failing because it is trying to find the docs in the repo pulled from the main branch. Since you have pushed the changes to your branch, you need to update the repo source to pull the docs from your branch instead.
git clone https://github.com/prometheus-operator/prometheus-operator -b main --depth 1 repos/prometheus-operator
For now, you need to change it to your branch, something like this temporarily:
git clone https://github.com/Dharma-09/prometheus-operator -b main --depth 1 repos/prometheus-operator
After your PR in the main repo is merged, we can then change the source repo back to the main branch of prometheus-operator and it will work fine. |
|
@Dharma-09 Can you change the source path in the synchronization file back to the original? With this, the page will be added to the website. |
|
Thanks @Dharma-09 |
This pull request updates the synchronization.sh file in the website repository to include the new Prometheus & Alertmanager Exposure Guide.
Here is the first preview, confirming that the new documentation page is working as expected.
The remaining screenshots state that the broken links have been fixed and no longer result in 404 errors.