Skip to content

ci: Remove dead submodules checkout option - #6448

Closed
runningcode wants to merge 1 commit into
mainfrom
no/ci-remove-dead-submodules
Closed

runningcode wants to merge 1 commit into
mainfrom
no/ci-remove-dead-submodules

Conversation

@runningcode

Copy link
Copy Markdown
Contributor

📜 Description

Remove submodules: 'recursive' from the actions/checkout steps in 11 CI workflows. Where it was the only input, the empty with: block goes too.

💡 Motivation and Context

The repository has had no .gitmodules since the NDK moved to sentry-native in #3189, so the option does nothing and suggests submodules still exist.

💚 How did you test it?

CI on this PR.

📝 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

🤖 Generated with Claude Code

#skip-changelog

The repository has no submodules since the NDK moved to sentry-native
in #3189, so `submodules: 'recursive'` on actions/checkout does nothing.

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 closed this Nov 2, 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.

1 participant