ADFA-2501 Fix name instead of it.name

[hal-eisen-adfa]

No description provided.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Release Notes

• Fixed task dependency condition in Gradle build configuration: Changed the lint task configuration in app/build.gradle.kts to use the task's implicit name property instead of explicit it.name reference when checking for "release" tasks in the afterEvaluate block
  • Affects: V8 and V7 lint-matching task blocks
  • Change: if (it.name.lowercase().contains("release")) → if (name.lowercase().contains("release"))
  • Impact: Release-specific dependency wiring on bundlePluginArtifactsForRelease now relies on implicit receiver semantics

Risk/Best Practices Notes

• ⚠️Implicit Receiver Dependency: The change removes explicit task reference (it) in favor of relying on implicit this receiver within the configureEach block. While valid, this reduces code clarity and could be harder to debug if scope becomes ambiguous
• ⚠️Behavioral Consistency: Both V8 and V7 lint task configurations were updated consistently, but verify that release tasks are still correctly identified by lowercase name matching after this change
• ✅ No Breaking Changes: The fix maintains functional equivalence for properly scoped task configurations

Walkthrough

The build configuration logic in app/build.gradle.kts has been refactored to check the task's name directly instead of referencing the task object when determining release-specific dependency wiring for lint tasks (V7 and V8).

Changes

Cohort / File(s)	Summary
Build Configuration Logic app/build.gradle.kts	Modified afterEvaluate release-detection logic: changed from task object reference (it) to task name reference (name.lowercase().contains("release")) for both V8 and V7 lint-matching blocks, affecting bundlePluginArtifactsForRelease dependency resolution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• ADFA-2496: fix: resolve Gradle implicit dependency validation for lint tasks #789: Also modifies afterEvaluate wiring for V7/V8 lint tasks' dependencies; complements this PR's release-name check logic with bundle-assets dependency handling.

Suggested reviewers

• jatezzz

Poem

🐰 A task name whispers true,
No object reference needed, shiny and new!
Release detection flows with grace,
Gradle's build logic finds its place! ✨

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c56d41e and 41c056c.

📒 Files selected for processing (1)

• app/build.gradle.kts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}