Skip to content

#570 FIX broken links#644

Open
RamakrishnanPK wants to merge 3 commits into
eclipse-score:mainfrom
bgsw-contrib:bgsw-develop
Open

#570 FIX broken links#644
RamakrishnanPK wants to merge 3 commits into
eclipse-score:mainfrom
bgsw-contrib:bgsw-develop

Conversation

@RamakrishnanPK

Copy link
Copy Markdown
Contributor

[TASK] Add broken link fix extension to ignore errors

  • Correct the broken links to the latest links available.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 352ddfeb-8ae5-410f-88e3-4e1b16f98573
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 7.201s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@RamakrishnanPK

Copy link
Copy Markdown
Contributor Author

Hi @MaximilianSoerenPollak,

Request to review and provide your comments.

Currently the checks were failing due issues on other repos via ##634
Create a ticket to fix the issue with process_description repo: eclipse-score/process_description#749

@MaximilianSoerenPollak

Copy link
Copy Markdown
Contributor

I'm not sure if I like the fix being a sphinx extension but I honestly do not have a better idea here.

@RamakrishnanPK

Copy link
Copy Markdown
Contributor Author

I'm not sure if I like the fix being a sphinx extension but I honestly do not have a better idea here.

1. 403 client errors - due to server not allowing bot requests

user_agent

user_agent = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36'
User agent configuration did not work!

linkcheck_ignore

linkcheck_ignore = [
    r'https://example\.com/broken-page',  # Ignores a specific URL
]

but this will ignore any real broken links as well. ( I mean 404 errors). I have to only ignore check only for 403 errors.

2. .md file referenced in the documentation with anchors.

Will have to wait until those are published / released as .html pages. but this is not in our control.

So, I do not have an option either! Urgency is BOT would again create a "Broken Links" automated issue!

Comment thread BUILD
Comment on lines +43 to +46
alias(
name = "link_check",
actual = ":docs_link_check",
)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isnt this alias just confusing? None of the other docs targets have an alias.

@AlexanderLanin

Copy link
Copy Markdown
Member

does this replace .github/workflows/link_check.yml?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants