Skip to content

ci: Remove the dead submodules checkout option - #6447

Merged
runningcode merged 1 commit into
mainfrom
no/remove-dead-submodules-checkout
Nov 3, 2026
Merged

runningcode merged 1 commit into
mainfrom
no/remove-dead-submodules-checkout

Conversation

@runningcode

Copy link
Copy Markdown
Contributor

📜 Description

Removes submodules: 'recursive' from the actions/checkout steps in 11 workflows, along with any with: block that ends up empty. It also deletes the empty .gitmodules file.

The repository has no submodules: git ls-tree -r on main finds no gitlink entries, and nothing reads .gitmodules. The only submodule, sentry-android-ndk/sentry-native, went away when the NDK moved to sentry-native in #3189. Since then the option has done nothing, and it keeps getting copied into new workflows.

Other checkout inputs don't change. build.yml keeps ref: ${{ github.event.pull_request.head.sha || github.sha }}, which its snapshot upload needs.

💡 Motivation and Context

Came up while reviewing the new Samples workflow in #6392, which had copied the option from build.yml.

💚 How did you test it?

actionlint passes on all changed workflows.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

#skip-changelog

🤖 Generated with Claude Code

The repository has had no submodules since the NDK moved to
sentry-native (#3189), which left `.gitmodules` empty. The
`submodules: 'recursive'` input on our checkout steps has been a no-op
since, so drop it along with the empty file.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sentry

sentry Bot commented Nov 2, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.62.0 (1) release

⚙️ sentry-android Build Distribution Settings

@runningcode
runningcode marked this pull request as ready for review November 2, 2026 14:53
@runningcode
runningcode requested review from a team and adinauer as code owners November 2, 2026 14:53
@runningcode
runningcode requested review from 0xadam-brown and markushi and removed request for a team November 2, 2026 14:53
@runningcode runningcode added the ship-it PR is ready to merge from a reviewer perspective label Nov 2, 2026
@runningcode
runningcode merged commit c608820 into main Nov 3, 2026
47 checks passed
@runningcode
runningcode deleted the no/remove-dead-submodules-checkout branch November 3, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ship-it PR is ready to merge from a reviewer perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants