Skip to content

Comments

fix(ci): multiple fixes#1549

Merged
imstevenpmwork merged 6 commits intomainfrom
fix/ci_multiple_fixes
Jul 20, 2025
Merged

fix(ci): multiple fixes#1549
imstevenpmwork merged 6 commits intomainfrom
fix/ci_multiple_fixes

Conversation

@imstevenpmwork
Copy link
Collaborator

@imstevenpmwork imstevenpmwork commented Jul 20, 2025 •
edited
Loading

Fixes and Improvements:

  1. Documentation File Structure

    • All .mdx files now reside in the docs/ folder, with the source code referencing them via symlinks.
    • Previously, some files in docs/ were symlinks pointing to original files in the source code, which prevented the documentation workflow from triggering (since changes weren’t directly in docs/).

  2. Image Tagging on PR Merge

    • When a PR is merged into main, the image is now tagged as latest instead of pr-{empty}.

  3. Test Suite Separation

    • The fast and full test suites have been decoupled for better clarity.
    • This change also ensures compliance with required status checks in the repository settings.

  4. GPU Tests in Community PRs

    • GPU tests are temporarily skipped for community-contributed PRs.
    • Reason: Running them requires building a Docker image and deploying it in our cluster, which could pose security risks.
    • For now, only the full CPU test suite is executed for external contributions.

@imstevenpmwork imstevenpmwork self-assigned this Jul 20, 2025
@imstevenpmwork imstevenpmwork added bug Something isn’t working correctly CI Issues related to the continuous integration pipeline labels Jul 20, 2025
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Jul 20, 2025

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

CarolinePascal
CarolinePascal previously approved these changes Jul 20, 2025
View reviewed changes
@imstevenpmwork imstevenpmwork changed the title Fix/ci multiple fixes fix(ci): multiple fixes Jul 20, 2025
@imstevenpmwork imstevenpmwork merged commit e88b30e into main Jul 20, 2025
12 of 13 checks passed
@imstevenpmwork imstevenpmwork deleted the fix/ci_multiple_fixes branch July 20, 2025 21:09
AdilZouitine pushed a commit that referenced this pull request Aug 10, 2025
@imstevenpmwork @AdilZouitine
fix(ci): multiple fixes (#1549)
40bf5d5 
* fix(ci): tag of image when pushing to main

* fix(docs): remove symlink in docs folder

* chore(docs): move .mdx files to docs/ folder

* chore(docs): create symlink to docs files

* chore(ci): de-couple fast and full test pipeline

* fix(ci): skip GPU Tests for community PRs
milong26 pushed a commit to milong26/lerobot_diy that referenced this pull request Aug 26, 2025
@imstevenpmwork
fix(ci): multiple fixes (huggingface#1549)
9bd8d36 
* fix(ci): tag of image when pushing to main

* fix(docs): remove symlink in docs folder

* chore(docs): move .mdx files to docs/ folder

* chore(docs): create symlink to docs files

* chore(ci): de-couple fast and full test pipeline

* fix(ci): skip GPU Tests for community PRs
Ricci084 pushed a commit to JeffWang987/lerobot that referenced this pull request Sep 5, 2025
@imstevenpmwork
fix(ci): multiple fixes (huggingface#1549)
1b15e82 
* fix(ci): tag of image when pushing to main

* fix(docs): remove symlink in docs folder

* chore(docs): move .mdx files to docs/ folder

* chore(docs): create symlink to docs files

* chore(ci): de-couple fast and full test pipeline

* fix(ci): skip GPU Tests for community PRs
BillmanH pushed a commit to BillmanH/lerobot that referenced this pull request Sep 7, 2025
@imstevenpmwork @BillmanH
fix(ci): multiple fixes (huggingface#1549)
adf85fc 
* fix(ci): tag of image when pushing to main

* fix(docs): remove symlink in docs folder

* chore(docs): move .mdx files to docs/ folder

* chore(docs): create symlink to docs files

* chore(ci): de-couple fast and full test pipeline

* fix(ci): skip GPU Tests for community PRs
sandhya-cb pushed a commit to sandhya-cb/lerobot-clutterbot that referenced this pull request Jan 28, 2026
@imstevenpmwork @sandhya-cb
fix(ci): multiple fixes (huggingface#1549)
b67a0a6 
* fix(ci): tag of image when pushing to main

* fix(docs): remove symlink in docs folder

* chore(docs): move .mdx files to docs/ folder

* chore(docs): create symlink to docs files

* chore(ci): de-couple fast and full test pipeline

* fix(ci): skip GPU Tests for community PRs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CarolinePascal CarolinePascal CarolinePascal approved these changes

Assignees

@imstevenpmwork imstevenpmwork

Labels

bug Something isn’t working correctly CI Issues related to the continuous integration pipeline

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@imstevenpmwork @HuggingFaceDocBuilderDev @CarolinePascal