Skip to content

fix: Replace broken .claude skill symlink with correct relative path#6368

Open
Srihari1192 wants to merge 1 commit intofeast-dev:masterfrom
Srihari1192:fixClaudeSkillSymLinkPath
Open

fix: Replace broken .claude skill symlink with correct relative path#6368
Srihari1192 wants to merge 1 commit intofeast-dev:masterfrom
Srihari1192:fixClaudeSkillSymLinkPath

Conversation

@Srihari1192
Copy link
Copy Markdown
Contributor

@Srihari1192 Srihari1192 commented May 5, 2026

What this PR does / why we need it:

The .claude/skills/feast-user-guide/SKILL.md file was committed as a symlink with an incorrect relative path that resolves outside the repository boundary, causing the CI build pipeline's mandatory symlink security check to fail .

Which issue(s) this PR fixes:

Checks

  • I've made sure the tests are passing.
  • My commits are signed off (git commit -s)
  • My PR title follows conventional commits format

Testing Strategy

  • Unit tests
  • Integration tests
  • Manual tests
  • Testing is not required for this change

Misc

Signed-off-by: Srihari <svenkata@redhat.com>
@Srihari1192 Srihari1192 force-pushed the fixClaudeSkillSymLinkPath branch from b91a6df to 2c6dc55 Compare May 5, 2026 12:14
@Srihari1192 Srihari1192 changed the title fix: replace broken .claude skill symlink with correct relative path fix: Replace broken .claude skill symlink with correct relative path May 5, 2026
Copy link
Copy Markdown
Collaborator

@jyejare jyejare left a comment

Choose a reason for hiding this comment

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

Could you please check all other files for/if similar changes required original PR from @ntkathole .

@Srihari1192
Copy link
Copy Markdown
Contributor Author

@jyejare Yes Already looked other files. Changes looks good there

@Srihari1192 Srihari1192 requested a review from jyejare May 5, 2026 12:28
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.

2 participants