Skip to content

Migrate remaining /index URIs to /SKILL in universe/SKILL.md#11

Open
Berkkirik wants to merge 1 commit intoroboflow:mainfrom
Berkkirik:fix/universe-skill-uri
Open

Migrate remaining /index URIs to /SKILL in universe/SKILL.md#11
Berkkirik wants to merge 1 commit intoroboflow:mainfrom
Berkkirik:fix/universe-skill-uri

Conversation

@Berkkirik
Copy link
Copy Markdown

Summary

skills/universe/SKILL.md still uses the legacy /index suffix on roboflow:// resource URIs. PR #7 migrated this exact pattern (/index/SKILL) in skills/product-navigation/features-by-page.md after the index.mdSKILL.md rename in 7db9842, but missed two siblings in universe/SKILL.md. This applies the same fix.

Before

## Related Skills

- `roboflow://skills/data-management/index` — managing datasets after import
- `roboflow://skills/training-and-evaluation/index` — training on forked data

After

## Related Skills

- `roboflow://skills/data-management/SKILL` — managing datasets after import
- `roboflow://skills/training-and-evaluation/SKILL` — training on forked data

Repro

grep -rn '/index' skills/ --include='*.md'
# skills/universe/SKILL.md:150:- `roboflow://skills/data-management/index` — managing datasets after import
# skills/universe/SKILL.md:151:- `roboflow://skills/training-and-evaluation/index` — training on forked data

After this PR: no matches.

Test plan

  • grep -rn '/index' skills/ --include='*.md' returns no matches.
  • Pattern matches the canonical form used in features-by-page.md:116 (roboflow://skills/plans-and-pricing/SKILL).
  • Both referenced targets exist: skills/data-management/SKILL.md and skills/training-and-evaluation/SKILL.md.

PR roboflow#7 migrated `roboflow://skills/<name>/index` cross-references to
`roboflow://skills/<name>/SKILL` after the `index.md` → `SKILL.md` rename
in 7db9842, but missed two siblings in `universe/SKILL.md`. This applies
the same fix to bring the file in line with the rest of the repo.
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.

1 participant