Merge branch 'main' into copilot/fix-78

Author: trask

.github/workflows/build.yml

@@ -1,9 +1,9 @@
 name: Build
 
 on:
-  push:
-    branches:
-      - main
+  # push:
+  #   branches:
+  #     - main
   pull_request:
   workflow_dispatch:
 

.gitignore

@@ -1,6 +1,7 @@
 # Gradle
 /.gradle/
 /buildSrc/.gradle/
+/buildSrc/.kotlin/
 build/
 
 # Intellij

CONTRIBUTING.md

@@ -36,13 +36,13 @@ from the OpenTelemetry Java Instrumentation project.
 See the [style guide](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/contributing/style-guideline.md).
 from the OpenTelemetry Java Instrumentation project.
 
-## Running a smoke test in Intellij
+## Running a smoke test in IntelliJ
 
 Pick the smoke test that you want to run, e.g. `MongoTest`.
 If the smoke test is abstract, then pick one of its subclasses, e.g. `MongoTest$Tomcat8Java8Test`.
 Right-click on it and select run.
 
-## Debugging a smoke test in Intellij
+## Debugging a smoke test in IntelliJ
 
 Add `-PsmokeTestRemoteDebug=true` to the gradle args, e.g.
 
@@ -52,7 +52,7 @@ Then create a Remote JVM Debug configuration, e.g.
 
 ![Create a Remote JVM Debug configuration](docs/intellij-debug-smoke-test-part-2.png)
 
-Then run (not debug) the smoke test as above in [Running the smoke tests from Intellij](#running-a-smoke-test-in-intellij).
+Then run (not debug) the smoke test as above in [Running the smoke tests from IntelliJ](#running-a-smoke-test-in-intellij).
 
 The JVM inside the container will wait remote debugger to be attached before starting.
 

README.md

@@ -1,8 +1,6 @@
-[![Build Status](https://github-private.visualstudio.com/microsoft/_apis/build/status/CDPX/applicationinsights-java/applicationinsights-java-Windows-Buddy-master?branchName=refs%2Fpull%2F1583%2Fmerge)](https://github-private.visualstudio.com/microsoft/_build/latest?definitionId=224&branchName=refs%2Fpull%2F1583%2Fmerge)
-
 # Application Insights for Java
 
-See documentation at https://docs.microsoft.com/en-us/azure/azure-monitor/app/java-in-process-agent.
+See the [official documentation](https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-enable?tabs=java).
 
 ## If you need to build locally or want to contribute
 

agent/agent-profiler/request-triggers.md

@@ -81,7 +81,7 @@ Each individual configuration is formed of:
   supported type).
 - `profileDuration` - The duration in seconds of the profile to be collected when this SLA is
   breached.
-- `threshold` - The threshould applied to the output of the aggregation, if this value is breached a
+- `threshold` - The threshold applied to the output of the aggregation, if this value is breached a
   profile will be triggered, i.e the `breach-ratio` aggregation outputs the percentage of requests
   that breach the SLA, a threshold of 0.95 would trigger a profile if 95% of requests breach the
   SLA.

agent/runtime-attach/src/main/java/com/microsoft/applicationinsights/attach/ApplicationInsights.java

@@ -22,8 +22,8 @@
 public final class ApplicationInsights {
 
   /**
-   * This property allows configuring an Application Insights json file. It can be helpful to get an
-   * Application Insights json file by Spring profile.
+   * This property allows configuring an Application Insights JSON file. It can be helpful to load
+   * different Application Insights JSON files based on Spring profile.
    */
   public static final String APPLICATIONINSIGHTS_RUNTIME_ATTACH_CONFIGURATION_FILE =
       "applicationinsights.runtime-attach.configuration.classpath.file";
@@ -42,7 +42,7 @@ private ApplicationInsights() {}
    * Attach the Application Insights agent for Java to the current JVM. The attachment must be
    * requested at the beginning of the main method.
    *
-   * @throws ConfigurationException If the file given by the
+   * @throws ConfigurationException if the file specified by the
    *     applicationinsights.runtime-attach.configuration.classpath.file property was not found
    */
   public static void attach() {

classic-sdk/core/src/main/java/com/microsoft/applicationinsights/telemetry/AvailabilityTelemetry.java

@@ -63,7 +63,7 @@ public void setMessage(String message) {
     data.setMessage(message);
   }
 
-  /** Gets a dictionary of custom defined metrics. */
+  /** Gets a map of custom defined metrics. */
   public ConcurrentMap<String, Double> getMetrics() {
     return data.getMeasurements();
   }

classic-sdk/core/src/main/java/com/microsoft/applicationinsights/telemetry/BaseTelemetry.java

@@ -39,7 +39,7 @@ public TelemetryContext getContext() {
   }
 
   /**
-   * Gets a dictionary of application-defined property names and values providing additional
+   * Gets a map of application-defined property names and values providing additional
    * information about this event.
    */
   @Override

classic-sdk/core/src/main/java/com/microsoft/applicationinsights/telemetry/EventTelemetry.java

@@ -40,7 +40,7 @@ public void setName(String name) {
     data.setName(name);
   }
 
-  /** Gets a dictionary of custom defined metrics. */
+  /** Gets a map of custom defined metrics. */
   public ConcurrentMap<String, Double> getMetrics() {
     return data.getMeasurements();
   }

classic-sdk/core/src/main/java/com/microsoft/applicationinsights/telemetry/ExceptionTelemetry.java

@@ -73,7 +73,7 @@ public SeverityLevel getSeverityLevel() {
         : SeverityLevel.values()[data.getSeverityLevel().getValue()];
   }
 
-  /** Gets a dictionary of custom defined metrics. */
+  /** Gets a map of custom defined metrics. */
   public ConcurrentMap<String, Double> getMetrics() {
     return data.getMeasurements();
   }