Skip to content

[1/N] Remove skyrl folder on main#1138

Merged
erictang000 merged 2 commits into
NovaSky-AI:mainfrom
erictang000:remove_skyrl_main
Feb 16, 2026
Merged

[1/N] Remove skyrl folder on main#1138
erictang000 merged 2 commits into
NovaSky-AI:mainfrom
erictang000:remove_skyrl_main

Conversation

@erictang000
Copy link
Copy Markdown
Collaborator

@erictang000 erictang000 commented Feb 16, 2026

Step 1 in recovering the git history for all skyrl-train and skyrl-tx files. Remove all skyrl files, since we will move them all up to the top level now.

See #1139, #1140, #1141 for follow ups.


Open with Devin

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the skyrl directory and all its contents as the first step of a larger refactoring to move these files to the top level of the repository. The changes consist entirely of file deletions, which aligns with the stated goal. There are no code modifications to review in this PR. The changes look correct as part of the described refactoring plan.

@erictang000 erictang000 merged commit b29448c into NovaSky-AI:main Feb 16, 2026
2 of 5 checks passed
@erictang000 erictang000 deleted the remove_skyrl_main branch February 16, 2026 01:33
@erictang000 erictang000 changed the title [1/3] Remove skyrl folder on main [1/N] Remove skyrl folder on main Feb 16, 2026
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