Skip to content

Conversation

@JosephSanjaya
Copy link
Collaborator

πŸ“± Description

Integrate buildlogic for detekt and test

Platform

  • Android
  • iOS
  • Games-Unity
  • DevOps (AWS)
  • Website
  • C/Golang

🎯 Type of Change

  • πŸ› Bug fix
  • ✨ New feature
  • πŸ”§ Refactoring (code change that neither fixes a bug nor adds a feature)
  • πŸ“š Documentation update
  • 🎨 UI/UX improvement
  • ⚑ Performance improvement
  • πŸ§ͺ Test addition or improvement

πŸ“‹ Changes

πŸ”— Related Issues

πŸ§ͺ Tests Status

  • Tests ran successfully locally?
  • Added more tests? How many?
  • Code coverage percentage of the codebase: __%

🎯 Reviewers

@kcw-grunt, @josikie

@JosephSanjaya JosephSanjaya self-assigned this Sep 20, 2025
@JosephSanjaya JosephSanjaya force-pushed the js/feat/build-logic-test branch 2 times, most recently from 0194733 to 08ae6a4 Compare September 20, 2025 10:27
Copy link
Collaborator

@kcw-grunt kcw-grunt left a comment

Choose a reason for hiding this comment

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

Rough review as the linter is fixing .

@JosephSanjaya
Copy link
Collaborator Author

Rough review as the linter is fixing .

Yup agreed, this is baseline file, will try to reduce the chnges

@JosephSanjaya JosephSanjaya marked this pull request as draft September 21, 2025 04:12
@JosephSanjaya
Copy link
Collaborator Author

I will convert this to draft first, please check https://github.com/gruntsoftware/android-games/pull/5

this PR so i can use latest commit hash for this PR later.

@kcw-grunt @josikie

@JosephSanjaya
Copy link
Collaborator Author

Rough review as the linter is fixing .

I tried to modify the configuration but turn out the flow in detekt Baseline is:

  • Trigger formatting lint first
  • then write baseline config. that way it stilll causing a lot of formatter. πŸ˜‘πŸ˜‘

@JosephSanjaya JosephSanjaya force-pushed the js/feat/build-logic-test branch from 08ae6a4 to ddfff43 Compare September 22, 2025 10:19
@JosephSanjaya JosephSanjaya marked this pull request as ready for review September 22, 2025 10:35
Integrates the `gruntsoftware-build-logic` Git submodule into the project.

This change includes:
- Adding the submodule to `.gitmodules`.
- Including the build logic in `settings.gradle.kts`.
- Applying build logic plugins (`test` and `detekt`) in `app/build.gradle.kts`.
This commit removes the ktlint configuration and associated tasks from the `app/build.gradle.kts` file. It also removes the `pinterest-ktlint` dependency from the `gradle/libs.versions.toml` file.
@JosephSanjaya JosephSanjaya force-pushed the js/feat/build-logic-test branch from ddfff43 to b37c385 Compare September 22, 2025 10:41
@kcw-grunt kcw-grunt self-requested a review September 22, 2025 12:03
Copy link
Collaborator

@kcw-grunt kcw-grunt left a comment

Choose a reason for hiding this comment

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

lgtm

@kcw-grunt kcw-grunt merged commit 4a27ae0 into develop Sep 22, 2025
2 checks passed
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