Skip to content

Conversation

@chensuyue
Copy link
Contributor

@chensuyue chensuyue commented Nov 3, 2025

User description

Type of Change

validation job with GHA

Description

Add pr hyperlink and relative link scan job.
Scan all the pr updated readme, raise invalid link.


PR Type

Enhancement


Description

  • Add GitHub Actions workflow for PR link scanning

  • Scan PR updated README for invalid links


Diagram Walkthrough

flowchart LR
  A["Add GitHub Actions workflow"] -- "Scans PR README" --> B["Identify invalid links"]
Loading

File Walkthrough

Relevant files
Enhancement
README.md
Update LLMs Recipes link                                                                 

README.md

  • Update LLMs Recipes link to relative path
+1/-1     
pr-link-scan.yml
Add PR link scan job                                                                         

.github/workflows/pr-link-scan.yml

  • Add new workflow for PR link scanning
+157/-0 

Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
@PRAgent4INC
Copy link
Collaborator

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Possible Issue

The colspan attribute value is set to 1 for the LLMs Recipes link, which might be incorrect if the table structure expects a different colspan value.

<td colspan="1" align="center"><a href="./docs/source/llm_recipes.md">LLMs Recipes</a></td>

@PRAgent4INC
Copy link
Collaborator

PR Code Suggestions ✨

No code suggestions found for the PR.

chensuyue and others added 13 commits November 4, 2025 15:57
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
@chensuyue chensuyue requested review from thuang6 and xin3he November 6, 2025 02:48
@chensuyue
Copy link
Contributor Author

Those UT test failure are not related to this PR.

@chensuyue chensuyue merged commit 907c538 into master Nov 6, 2025
18 of 24 checks passed
@chensuyue chensuyue deleted the suyue/link_scan branch November 6, 2025 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants