Drop trl skills install and move the skill source out of the package

.github/workflows/sync-huggingface-skills.yml

@@ -50,14 +50,16 @@ jobs:
         uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d  # v10.0.1
 
       - name: Copy generated files
-        run: cp trl/skills/trl-training/SKILL.md skills-repo/skills/trl-training/
+        run: |
+          mkdir -p skills-repo/skills/trl-training
+          cp skills/trl-training/SKILL.md skills-repo/skills/trl-training/
 
       - name: Check for TRL skill changes
         id: check_changes
         working-directory: skills-repo
-        # git diff returns zero if there is no diff
+        # git status --porcelain prints nothing if there is no change
         run: |
-          if git diff --quiet -- skills/trl-training/SKILL.md; then
+          if [ -z "$(git status --porcelain -- skills/trl-training/)" ]; then
             echo "changed=false" >> "$GITHUB_OUTPUT"
             echo "No trl skill changes; skipping PR"
           else
@@ -82,7 +84,7 @@ jobs:
           body: |
             Auto-generated from [trl@${{ github.sha }}](https://github.com/huggingface/trl/commit/${{ github.sha }})
 
-            Triggered by changes to `trl/skills/trl-training`
+            Triggered by changes to `skills/trl-training`
 
             ---
             This PR was created automatically by the [sync-huggingface-skills](https://github.com/huggingface/trl/blob/main/.github/workflows/sync-huggingface-skills.yml) workflow.

MANIFEST.in

@@ -5,6 +5,5 @@ include trl/accelerate_configs/*.yaml
 include trl/templates/*.md
 include trl/chat_templates/*.jinja
 include trl/chat_templates/*.md
-include trl/skills/**/*.md
 recursive-exclude * __pycache__
 prune tests

pyproject.toml

@@ -169,7 +169,6 @@ extend-select = ["E", "F", "I", "W", "UP", "B", "T", "C"]
 "examples/**.py" = ["T201"]
 "scripts/**.py" = ["T201"]
 "trl/cli/**.py" = ["T201"]
-"trl/skills/cli.py" = ["T201"]
 # Ignore import violations in all `__init__.py` files.
 "__init__.py" = ["F401"]