Skip to content

feat: migrate appsflyer-kit from submodule to regular directory#650

Merged
denischilik merged 5 commits into
workstation/6.0-Releasefrom
feat/migrate-appsflyer
Mar 6, 2026
Merged

feat: migrate appsflyer-kit from submodule to regular directory#650
denischilik merged 5 commits into
workstation/6.0-Releasefrom
feat/migrate-appsflyer

Conversation

@denischilik

Copy link
Copy Markdown
Contributor

Background

Migrates AppsFlyer kit from git submodule to a regular directory in the main repo. The kit is relocated to kits/appsflyer/appsflyer-6 to support versioned kit structure (6 = major version of af-android-sdk).

What Has Changed

  • Migrated appsflyer-kit from submodule to regular directory
  • Relocated kit to kits/appsflyer/appsflyer-6
  • Updated settings-kits.gradle to reference new path (:kits:appsflyer:appsflyer-6)
  • Removed submodule entry from .gitmodules

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

@denischilik denischilik requested a review from a team as a code owner March 5, 2026 15:37
Comment thread kits/appsflyer/appsflyer-6/.gitignore Outdated
@denischilik denischilik marked this pull request as draft March 5, 2026 17:00
@denischilik denischilik marked this pull request as ready for review March 5, 2026 18:04
@sonarqubecloud

sonarqubecloud Bot commented Mar 5, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this is being used. Can we remove this, or capture this as a todo item to remove?

Comment thread .trunk/trunk.yaml
- gradlew
- "**/gradlew"
- scripts/install-start-emulator.sh
- kits/appsflyer/appsflyer-6/**

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please add a TODO item for the trunk bypasses we are making.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@thomson-t Added it to the cleanup.

@nickolas-dimitrakas nickolas-dimitrakas left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@denischilik denischilik merged commit 790dc05 into workstation/6.0-Release Mar 6, 2026
13 of 17 checks passed
@denischilik denischilik deleted the feat/migrate-appsflyer branch March 6, 2026 15:45
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.

3 participants