Skip to content

Merge bugfix/resource-file-config into main#5

Merged
CoderMP merged 1 commit intomainfrom
bugfix/resource-file-config
Aug 6, 2025
Merged

Merge bugfix/resource-file-config into main#5
CoderMP merged 1 commit intomainfrom
bugfix/resource-file-config

Conversation

@CoderMP
Copy link
Copy Markdown
Owner

@CoderMP CoderMP commented Aug 6, 2025

🚀 Description

PR refactors cleanup.gradle.kts to include replacement for postSplashScreenTheme property contained in splash.xml.

📄 Motivation and Context

PR corrects a logical error with renaming an XML property in splash.xml during project creation.

🧪 How Has This Been Tested?

📷 Screenshots (if appropriate)

📦 Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Version Bump (non-breaking change which bumps the version)
  • New feature (non-breaking change which adds functionality)
  • New utility (non-breaking change which supplements functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

…ScreenTheme property contained in `splash.xml`
@CoderMP CoderMP requested a review from Copilot August 6, 2025 19:29
@CoderMP CoderMP self-assigned this Aug 6, 2025
Copy link
Copy Markdown

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 fixes a logical error in the project template cleanup process by ensuring both the main theme name and the postSplashScreenTheme property in XML files are properly renamed during project creation.

  • Adds replacement logic for postSplashScreenTheme property to use the new project name
  • Improves code formatting by splitting long parameter lines for better readability

Comment thread buildSrc/src/main/kotlin/cleanup.gradle.kts
@CoderMP CoderMP merged commit c360f8e into main Aug 6, 2025
4 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.

2 participants