Skip to content

Conversation

@deerskindoll
Copy link
Contributor

What does this pull request change?

added gnu.org and prometheus.io links to ignoreURLs list for htmltest

What issues does this pull request fix or reference?

Specify the version of the product this pull request applies to

main

Pull Request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
    • The page contains a redirection for the previous URL.
    • Propagate the URL change in:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

@deerskindoll deerskindoll requested a review from a team as a code owner April 4, 2025 22:40
@github-actions
Copy link

github-actions bot commented Apr 4, 2025

🎊 Navigate the preview: https://67f0645f901b4b856c38b27d--eclipse-che-docs-pr.netlify.app 🎊

@github-actions
Copy link

github-actions bot commented Apr 4, 2025

Click here to review and test in web IDE: Contribute

@ibuziuk
Copy link
Member

ibuziuk commented Apr 22, 2025

@deerskindoll could you clarify which exact links are not accessible?

@deerskindoll
Copy link
Contributor Author

@deerskindoll could you clarify which exact links are not accessible?

these two pages cause issues:

https://prometheus.io/* (multiple pages)
https://www.gnu.org/* (multiple pages)

@ibuziuk
Copy link
Member

ibuziuk commented Apr 22, 2025

@deerskindoll right, but those links are accessible, I'm wondering why you want to exclude them in the first place?

@ibuziuk
Copy link
Member

ibuziuk commented Apr 22, 2025

@deerskindoll are there some dead links in our doc that should be updated?

@deerskindoll deerskindoll merged commit 1a6c443 into eclipse-che:main Apr 22, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants