Skip to content

Conversation

@xingyaoww
Copy link
Contributor

Summary

This PR removes all git submodules from the repository.

Changes

  • Removed agent-sdk submodule that was in an inconsistent state
    • The .gitmodules file was already missing
    • The gitlink (160000 mode entry) remained in the git index
    • This caused errors when running git submodule status

Verification

  • Confirmed the submodule was registered using git ls-files --stage | grep 160000
  • Removed using git rm --cached agent-sdk
  • Repository is now clean of all submodule references

@xingyaoww can click here to continue refining the PR

- Removed agent-sdk submodule that was in an inconsistent state
- The .gitmodules file was already missing but the gitlink remained

Co-authored-by: openhands <openhands@all-hands.dev>
@xingyaoww xingyaoww marked this pull request as ready for review October 21, 2025 15:31
@xingyaoww xingyaoww enabled auto-merge (squash) October 21, 2025 15:31
@xingyaoww xingyaoww requested a review from enyst October 21, 2025 15:31
@enyst
Copy link
Collaborator

enyst commented Oct 21, 2025

@OpenHands Merge main into this PR branch and fix the conflict.

@openhands-ai
Copy link

openhands-ai bot commented Oct 21, 2025

I'm on it! enyst can track my progress at all-hands.dev

Copy link
Collaborator

@enyst enyst left a comment

Choose a reason for hiding this comment

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

I had the impression we already deleted it

…gent-sdk removal and accepting upstream doc changes

- Keep deletion of agent-sdk (submodule/content) to align with PR purpose
- Accept updates from main to .openhands/microagents/repo.md

Co-authored-by: openhands <openhands@all-hands.dev>
@xingyaoww xingyaoww merged commit 15d07cf into main Oct 21, 2025
2 checks passed
@openhands-ai
Copy link

openhands-ai bot commented Oct 21, 2025

Final summary of work completed

What I did:

  • Merged main into the PR branch remove-git-submodules
  • Resolved the merge conflict on agent-sdk by keeping the deletion (aligned with the PR’s goal to remove submodules)
  • Accepted upstream updates to .openhands/microagents/repo.md from main
  • Committed and pushed the merge resolution

Verification:

  • No gitlinks remain in the index (no entries with mode 160000)
  • agent-sdk directory is removed from the tree
  • Only change introduced by the merge is updating .openhands/microagents/repo.md; no extraneous changes

Commit:

  • a57cdb1 — merge: main into remove-git-submodules; resolve conflict by keeping agent-sdk removal and accepting upstream doc changes

View full conversation

@mamoodi mamoodi deleted the remove-git-submodules branch October 22, 2025 14:51
enyst pushed a commit that referenced this pull request Nov 8, 2025
)

Co-authored-by: openhands <openhands@all-hands.dev>
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.

4 participants