fix(deps): update dependency com.google.guava:guava-bom to v33.4.5-jre - abandoned (#13555)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Lauri Tulmin <[email protected]>

Author: renovate[bot]

dependencyManagement/build.gradle.kts

@@ -29,7 +29,7 @@ val DEPENDENCY_BOMS = listOf(
 
   "com.fasterxml.jackson:jackson-bom:2.18.3",
   "com.squareup.okio:okio-bom:3.10.2", // see https://github.com/open-telemetry/opentelemetry-java/issues/5637
-  "com.google.guava:guava-bom:33.4.0-jre",
+  "com.google.guava:guava-bom:33.4.5-jre",
   "org.apache.groovy:groovy-bom:${groovyVersion}",
   "io.opentelemetry:opentelemetry-bom:${otelSdkVersion}",
   "io.opentelemetry:opentelemetry-bom-alpha:${otelSdkAlphaVersion}",

docs/contributing/style-guideline.md

@@ -108,18 +108,7 @@ All parameters and fields which can be `null` should be annotated with `@Nullabl
 (specifically `javax.annotation.Nullable`, which is included by the
 `otel.java-conventions` gradle plugin as a `compileOnly` dependency).
 
-There is no need to use `@NonNull`, as this is the default, which should be declared in a
-`package-info.java` file on the root package of each module, e.g.
-
-```java
-@DefaultQualifier(
-    value = NonNull.class,
-    locations = {TypeUseLocation.FIELD, TypeUseLocation.PARAMETER, TypeUseLocation.RETURN})
-package io.opentelemetry.instrumentation.api;
-
-import org.checkerframework.checker.nullness.qual.NonNull;
-import org.checkerframework.framework.qual.DefaultQualifier;
-```
+There is no need to use `@NonNull`, as this is the default.
 
 Public APIs should still defensively check for `null` parameters, even if the parameter is not
 annotated with `@Nullable`. Internal APIs do not need to defensively check for `null` parameters.