Skip to content

[REFACTOR] Gradle 9#157

Merged
alex-amenos merged 7 commits intoepic/migrate-to-agp-9-0from
refactor/agp9
Mar 17, 2026
Merged

[REFACTOR] Gradle 9#157
alex-amenos merged 7 commits intoepic/migrate-to-agp-9-0from
refactor/agp9

Conversation

@alex-amenos
Copy link
Owner

@alex-amenos alex-amenos commented Mar 17, 2026

⚡️ Proposed Changes

  • Gradle 9
  • Refactor initial loading state in sevetal ViewModels

ℹ️ Additional Info

  • Add any additional useful context or info

🔗 Related Links

  • Add helpful links for this pull request

✅ Checklist

  • Unit Tests
  • Integration Tests
  • Compose Tests
  • Screenshot Tests
  • Updated string
  • Manually tested

📷 Screenshots

Copilot AI review requested due to automatic review settings March 17, 2026 20:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR makes small formatting and configuration tweaks to improve code consistency and adjust ktlint behavior across the project.

Changes:

  • Wraps when branches and else handling in explicit blocks for clearer structure.
  • Simplifies a Compose Modifier chain formatting into a single line.
  • Updates repo config: removes an .opencode ignore entry and disables ktlint’s condition-wrapping rule.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
feature/my-playground/src/main/java/com/alxnophis/jetpack/myplayground/ui/viewmodel/MyPlaygroundViewModel.kt Adds explicit block bodies to when branches in event handling.
feature/home/src/main/java/com/alxnophis/jetpack/home/ui/composable/HomeFeature.kt Wraps navigation and fallback handling into blocks for consistency.
feature/file-downloader/src/main/java/com/alxnophis/jetpack/filedownloader/ui/composable/FileDownloaderScreen.kt Collapses a multi-line Modifier chain into a single line.
.gitignore Removes ignore entry for /.opencode/.
.editorconfig Disables ktlint standard:condition-wrapping rule.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@alex-amenos alex-amenos merged commit a079ec0 into epic/migrate-to-agp-9-0 Mar 17, 2026
1 check passed
@alex-amenos alex-amenos deleted the refactor/agp9 branch March 17, 2026 22:04
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.

2 participants