Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: support both gradle and gradle.kts file formats #305 #349

Closed
wants to merge 1 commit into from

Conversation

yyyng2
Copy link

@yyyng2 yyyng2 commented Dec 29, 2024

Description

  • Add .kts extension check in androidAppBuildGradle path
  • Update regex patterns to match Kotlin DSL syntax
  • Handle both file formats when reading gradle contents
  • Prioritize checking .kts files before .gradle files (new projects are more likely to use .kts format)

Type of Change

  • feat -- New feature (non-breaking change which adds functionality)
  • 🛠️ fix -- Bug fix (non-breaking change which fixes an issue)
  • ! -- Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 refactor -- Code refactor
  • ci -- Build configuration change
  • 📝 docs -- Documentation
  • 🗑️ chore -- Chore

Sorry, something went wrong.

@CLAassistant
Copy link

CLAassistant commented Dec 29, 2024

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the Needs Attention OP created or responded to issue and it needs attention. label Dec 29, 2024
@yyyng2 yyyng2 changed the title fix: support both build.gradle and build.gradle.kts file formats #305 fix: support both gradle and gradle.kts file formats #305 Dec 29, 2024
Copy link

Hello 👋, this PR has been opened for more than 2 months with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 15 days until this gets closed automatically

@github-actions github-actions bot added the Stale label Jan 26, 2025
@github-actions github-actions bot closed this Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Attention OP created or responded to issue and it needs attention. Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants