Skip to content

Add base to root module to enable ./gradlew clean#505

Merged
jack-berg merged 1 commit into
open-telemetry:mainfrom
jack-berg:fix-build
Jun 17, 2026
Merged

Add base to root module to enable ./gradlew clean#505
jack-berg merged 1 commit into
open-telemetry:mainfrom
jack-berg:fix-build

Conversation

@jack-berg

Copy link
Copy Markdown
Member

If you've got the project checked out locally, and try to run the commands from CONTRIBUTING.md after a semconv version update, then gradle uses the existing semconv download from build/ instead of downloading a new one:

./gradlew clean generateSemanticConventions --console=plain
./gradlew spotlessApply

There is probably more work to do with properly specifying gradle task input/outputs such that when var semanticConventionsVersion = 1.42.0 and only build/semantic-conventions-1.41.1 exists, the downloadSemanticConventions task knows it needs to download build/semantic-conventions-1.42.0. But the simplest / immediate fix is to make sure the clean task from ./gradlew clean generateSemanticConventions --console=plain actually deletes build/ and everything in it.

@jack-berg jack-berg requested review from a team as code owners June 17, 2026 19:55
@jack-berg jack-berg merged commit 563435a into open-telemetry:main Jun 17, 2026
14 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