Skip to content

Conversation

@cartland
Copy link
Contributor

The ./gradlew spotlessApply command was failing in the :watchfacepush:validator and :identity:credentialmanager modules due to incorrect path calculations during the format and apply steps. This was caused by ambiguous source directory configurations in their build.gradle.kts files, which led to errors like NoSuchFileException and FileAlreadyExistsException.

This commit resolves these issues by explicitly defining the sourceSets in the build scripts for the affected modules. This removes the ambiguity and ensures that Gradle and the Spotless plugin can reliably locate the source files.

As a result of this fix, the apply_spotless.yml GitHub workflow has been simplified to run a single, project-wide spotlessApply command.

The .gitignore has also been updated to correctly ignore all build directories.

cartland and others added 2 commits August 15, 2025 11:38
The `./gradlew spotlessApply` command was failing in the
`:watchfacepush:validator` and `:identity:credentialmanager` modules
due to incorrect path calculations during the format and apply steps.
This was caused by ambiguous source directory configurations in their
`build.gradle.kts` files, which led to errors like `NoSuchFileException` and `FileAlreadyExistsException`.

This commit resolves these issues by explicitly defining the
`sourceSets` in the build scripts for the affected modules.
This removes the ambiguity and ensures that Gradle and the
Spotless plugin can reliably locate the source files.

As a result of this fix, the `apply_spotless.yml` GitHub workflow has
been simplified to run a single, project-wide `spotlessApply` command.

The `.gitignore` has also been updated to correctly ignore all `build` directories.
@snippet-bot
Copy link

snippet-bot bot commented Aug 15, 2025

Here is the summary of changes.

You are about to add 23 region tags.
You are about to delete 23 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@cartland cartland merged commit 39218cc into main Aug 18, 2025
5 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.

3 participants