Skip to content

fix: debug cleanup#1

Open
maxwellt7 wants to merge 1 commit into
masterfrom
fix/debug-cleanup
Open

fix: debug cleanup#1
maxwellt7 wants to merge 1 commit into
masterfrom
fix/debug-cleanup

Conversation

@maxwellt7
Copy link
Copy Markdown
Owner

Summary

  • Gradle 2.10 → 7.6.4: Old version couldn't parse Java 17 version string
  • jcenter() → mavenCentral(): jcenter is sunset/read-only
  • Conditional private repo/plugins: Build no longer crashes without SmartThings Artifactory credentials — private deps and deployment plugins activate only when smartThingsArtifactoryUserName is set in gradle.properties
  • Compile → Implementation: Migrated deprecated dependency configurations
  • Fixed configure/unconfigure tasks: Original Exec tasks had multiple commandLine calls but only the last one executed; now uses doLast with exec blocks so all commands run
  • Dynamic class loading for Slack classes: Avoids compile-time resolution of FileUpload/Message when private libs aren't on classpath
  • Added gradle.properties: Placeholder for credentials
  • Added .env.example: Documents all required configuration

Test plan

  • ./gradlew build succeeds without credentials (clean build)
  • ./gradlew tasks lists all expected tasks including devicetypesClasses and smartappsClasses
  • With valid SmartThings Artifactory credentials, private deps resolve and deployment plugins load

🤖 Generated with Claude Code

- Upgrade Gradle wrapper 2.10 -> 7.6.4 (Java 17 compatibility)
- Replace sunset jcenter() with mavenCentral()
- Make private SmartThings Maven repo and plugins conditional on credentials
- Migrate deprecated Compile configurations to Implementation
- Fix configure/unconfigure tasks: multiple commandLine calls in Exec only
  kept the last one; now uses doLast with exec blocks to run all commands
- Use dynamic class loading for SmartThings Slack classes to avoid
  compile-time resolution when private deps are unavailable
- Add gradle.properties with placeholder credentials
- Add .env.example documenting required configuration

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant