From ee36ce4b51967a6caa4aa24f55e3fb02c80a1bc5 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sat, 23 Aug 2025 15:22:01 -0700 Subject: [PATCH 01/10] Add copilot review instructions --- .github/copilot-instructions.md | 30 ++++++ CONTRIBUTING.md | 89 +++++++--------- README.md | 23 +---- docs/style-guide.md | 173 ++++++++++++++++++++++++++++++++ 4 files changed, 240 insertions(+), 75 deletions(-) create mode 100644 .github/copilot-instructions.md create mode 100644 docs/style-guide.md diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md new file mode 100644 index 000000000..f5c985dd5 --- /dev/null +++ b/.github/copilot-instructions.md @@ -0,0 +1,30 @@ +# Copilot Instructions for OpenTelemetry Java Contrib + +This repository provides observability instrumentation for Java applications. + +## Code Review Priorities + +### Style Guide Compliance + +**PRIORITY**: Verify that all code changes follow the [Style Guide](../docs/style-guide.md). Check: + +- Code formatting (auto-formatting, static imports, class organization) +- Java language conventions (`final` usage, `@Nullable` annotations, `Optional` usage) +- Performance constraints (hot path allocations) +- Implementation patterns (SPI registration, configuration conventions) +- Gradle conventions (Kotlin DSL, plugin usage, module naming) +- Documentation standards (README files, deprecation processes) + +### Critical Areas + +- **Public APIs**: Changes affect downstream users and require careful review +- **Performance**: Instrumentation must have minimal overhead +- **Thread Safety**: Ensure safe concurrent access patterns +- **Memory Management**: Prevent leaks and excessive allocations + +### Quality Standards + +- Proper error handling with appropriate logging levels +- OpenTelemetry specification and semantic convention compliance +- Resource cleanup and lifecycle management +- Comprehensive unit tests for new functionality diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 6d8a6a806..04a50c300 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,82 +1,65 @@ # Contributing -Welcome to the OpenTelemetry Java Contrib Repository! +Welcome to the OpenTelemetry Java Contrib repository! ## Introduction -This repository focuses on providing tools and utilities for Java-based observability, such as remote JMX metric gathering and reporting. We’re excited to have you here! Whether you’re fixing a bug, adding a feature, or suggesting an idea, your contributions are invaluable. +This repository provides observability libraries and utilities for Java applications that complement +the [OpenTelemetry Java SDK](https://github.com/open-telemetry/opentelemetry-java) and +[OpenTelemetry Java Instrumentation](https://github.com/open-telemetry/opentelemetry-java-instrumentation) +projects. -Before submitting new features or changes to current functionality, it is recommended to first -[open an issue](https://github.com/open-telemetry/opentelemetry-java-contrib/issues/new) -and discuss your ideas or propose the changes you wish to make. - -Questions? Ask in the OpenTelemetry [java channel](https://cloud-native.slack.com/archives/C014L2KCTE3) +Before submitting new features or changes, please consider +[opening an issue](https://github.com/open-telemetry/opentelemetry-java-contrib/issues/new) first to +discuss your ideas. Pull requests for bug fixes are always welcome! -## Pre-requisites - -To work with this repository, ensure you have: - -### Tools: - -Java 17 or higher - -### Platform Notes: +## Building and Testing -macOS/Linux: Ensure JAVA_HOME is set correctly. +While most modules target Java 8, building this project requires Java 17 or higher. -## Workflow - -1. Fork the repository -2. Clone locally -3. Create a branch before working on an issue - -## Local Run/Build - -In order to build and test this whole repository you need JDK 11+. - -#### Snapshot builds - -For developers testing code changes before a release is complete, there are -snapshot builds of the `main` branch. They are available from -the Sonatype snapshot repository at `https://central.sonatype.com/repository/maven-snapshots/` -([browse](https://central.sonatype.com/service/rest/repository/browse/maven-snapshots/io/opentelemetry/contrib/)). - -#### Building from source - -Building using Java 11+: +To build the project: ```bash -$ java -version +./gradlew assemble ``` +To run the tests: + ```bash -$ ./gradlew assemble +./gradlew test ``` -## Testing +Some modules include integration tests that can be run with: ```bash -$ ./gradlew test +./gradlew integrationTest ``` -### Some modules have integration tests +## Snapshot Builds -``` -$ ./gradlew integrationTest -``` +Snapshot builds of the `main` branch are available from the Sonatype snapshot repository at: +`https://central.sonatype.com/repository/maven-snapshots/` +([browse](https://central.sonatype.com/service/rest/repository/browse/maven-snapshots/io/opentelemetry/contrib/)). + +## Style Guide + +See [Style Guide](docs/style-guide.md). -Follow the Java Instrumentation [Style Guide](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/contributing/style-guideline.md) from the opentelemetry-java-instrumentation repository. +## Pull Request Guidelines -Failure? Check logs for errors or mismatched dependencies. +When submitting a pull request, please ensure that you: -## Gradle conventions +- Clearly describe the change and its motivation +- Mention any breaking changes +- Include tests for new functionality +- Follow the [Style Guide](docs/style-guide.md) -- Use kotlin instead of groovy -- Plugin versions should be specified in `settings.gradle.kts`, not in individual modules -- All modules use `plugins { id("otel.java-conventions") }` +## Getting Help -## Further Help +If you need assistance or have questions: -Join [#otel-java](https://cloud-native.slack.com/archives/C014L2KCTE3) on OpenTelemetry Slack +- Post on the [#otel-java](https://cloud-native.slack.com/archives/C014L2KCTE3) Slack channel +- [Open an issue](https://github.com/open-telemetry/opentelemetry-java-contrib/issues/new/choose) in + this repository diff --git a/README.md b/README.md index c42ea7e27..29b2e01f2 100644 --- a/README.md +++ b/README.md @@ -48,30 +48,9 @@ On reaching stable status, the `otel.stable` value in `gradle.properties` should Note that currently all the libraries are released together with the version of this repo, so breaking changes (after stable status is reached) would bump the major version of all libraries together. This could get complicated so `stable` has a high bar. -## Getting Started - -```bash -# Apply formatting -$ ./gradlew spotlessApply - -# Build the complete project -$ ./gradlew build - -# Run integration tests -$ ./gradlew integrationTest - -# Clean artifacts -$ ./gradlew clean -``` - ## Contributing -The Java Contrib project was initially formed to provide methods of easy remote JMX metric gathering and reporting, -which is actively in development. If you have an idea for a similar use case in the metrics, traces, or logging -domain we would be very interested in supporting it. Please -[open an issue](https://github.com/open-telemetry/opentelemetry-java-contrib/issues/new/choose) to share your idea or -suggestion. PRs are always welcome and greatly appreciated, but for larger functional changes a pre-coding introduction -can be helpful to ensure this is the correct place and that active or conflicting efforts don't exist. +See [CONTRIBUTING.md](CONTRIBUTING.md). ### Maintainers diff --git a/docs/style-guide.md b/docs/style-guide.md new file mode 100644 index 000000000..8bf101218 --- /dev/null +++ b/docs/style-guide.md @@ -0,0 +1,173 @@ +# Style Guide + +This project follows the +[Google Java Style Guide](https://google.github.io/styleguide/javaguide.html). + +## Code Formatting + +### Auto-formatting + +The build will fail if source code is not formatted according to Google Java Style. + +Run the following command to reformat all files: + +```bash +./gradlew spotlessApply +``` + +For IntelliJ users, an `.editorconfig` file is provided that IntelliJ will automatically use to +adjust code formatting settings. However, it does not support all required rules, so you may still +need to run `./gradlew spotlessApply` periodically. + +### Static imports + +Consider statically importing the following commonly used methods and constants: + +- **Test methods** + - `io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions` (assertThat, satisfies, etc.) + - `org.assertj.core.api.Assertions.*` (assertThat, assertThatThrownBy, entry, etc.) + - `org.mockito.Mockito.*` (when, mock, verify, times, etc.) + - `org.mockito.ArgumentMatchers.*` (any, eq, anyLong, etc.) +- **Utility methods** + - `io.opentelemetry.api.common.AttributeKey.*` (stringKey, longKey, etc.) + - `java.util.Arrays.*` (asList, etc.) + - `java.util.Collections.*` (singletonList, emptyList, emptyMap, emptySet, etc.) + - `java.util.stream.Collectors.*` (toList, toMap, toSet, etc.) + - `java.util.Objects` (requireNonNull, etc.) + - `java.util.logging.Level.*` (FINE, INFO, WARNING, etc.) + - `java.nio.charset.StandardCharsets.*` (UTF_8, etc.) +- **Time unit constants** + - `java.util.concurrent.TimeUnit.*` (SECONDS, etc.) +- **OpenTelemetry semantic convention constants** + - All constants under `io.opentelemetry.semconv.**` + +### Class organization + +Prefer this order: + +- Static fields (final before non-final) +- Instance fields (final before non-final) +- Constructors +- Methods +- Nested classes + +**Method ordering**: Place calling methods above the methods they call. For example, place private +methods below the non-private methods that use them. + +**Static utility classes**: Place the private constructor (used to prevent instantiation) after all +methods. + +## Java Language Conventions + +### Package conventions + +Classes in `.internal` packages are not considered public API and may change without notice. These +packages contain implementation details that should not be used by external consumers. + +- Use `.internal` packages for implementation classes that need to be public within the module but + should not be used externally +- Try to avoid referencing `.internal` classes from other modules + +### `final` keyword usage + +Public API classes should be declared `final` where possible. + +Methods should only be declared `final` if they are in non-final public API classes. + +Fields should be declared `final` where possible. + +Method parameters and local variables should never be declared `final`. + +### `@Nullable` annotation usage + +**Note: This section is aspirational and may not reflect the current codebase.** + +Annotate all parameters and fields that can be `null` with `@Nullable` (specifically +`javax.annotation.Nullable`, which is included by the `otel.java-conventions` Gradle plugin as a +`compileOnly` dependency). + +`@NonNull` is unnecessary as it is the default. + +**Defensive programming**: Public APIs should still check for `null` parameters even if not +annotated with `@Nullable`. Internal APIs do not need these checks. + +**Enforcement**: Use the `otel.errorprone-conventions` Gradle plugin in all modules: + +```kotlin +plugins { + id("otel.errorprone-conventions") +} +``` + +### `Optional` usage + +Following the reasoning from +[Writing a Java library with better experience (slide 12)](https://speakerdeck.com/trustin/writing-a-java-library-with-better-experience?slide=12), +`java.util.Optional` usage is kept to a minimum. + +**Guidelines**: + +- `Optional` shouldn't appear in public API signatures +- Avoid `Optional` on the hot path (instrumentation code), unless the instrumented library uses it + +## Tooling conventions + +### AssertJ + +Prefer AssertJ assertions over JUnit assertions (assertEquals, assertTrue, etc.) for better error +messages. + +### AutoService + +Use the `@AutoService` annotation when implementing SPI interfaces. This automatically generates the +necessary `META-INF/services/` files at compile time, eliminating the need to manually create and +maintain service registration files. + +```java +@AutoService(AutoConfigurationCustomizerProvider.class) +public class MyCustomizerProvider implements AutoConfigurationCustomizerProvider { + // implementation +} +``` + +### Gradle + +- Use Kotlin instead of Groovy for build scripts +- Plugin versions should be specified in `settings.gradle.kts`, not in individual modules +- All modules should use `plugins { id("otel.java-conventions") }` +- Set module names with `otelJava.moduleName.set("io.opentelemetry.contrib.mymodule")` + +## Configuration + +- Use `otel.` prefix for all configuration property keys +- Read config via the `ConfigProperties` interface +- Provide sensible defaults and document all options +- Validate configuration early with clear error messages + +## Performance + +Avoid allocations on the hot path (instrumentation code) whenever possible. This includes `Iterator` +allocations from collections; note that `for (SomeType t : plainJavaArray)` does not allocate an +iterator object. + +Non-allocating Stream API usage on the hot path is acceptable but may not fit the surrounding code +style; this is a judgment call. Some Stream APIs make efficient allocation difficult (e.g., +`collect` with pre-sized sink data structures involves convoluted `Supplier` code, or lambdas passed +to `forEach` may be capturing/allocating lambdas). + +## Documentation + +### Component README files + +- Include a component owners section in each module's README +- Document configuration options with examples + +### Deprecation and breaking changes + +Breaking changes are allowed in unstable modules (published with `-alpha` version suffix). + +1. Mark APIs with `@Deprecated` and a removal timeline (there must be at least one release with the + API marked as deprecated before removing it) +2. Document the replacement in Javadoc with `@deprecated` tag +3. Note the migration path for breaking changes under a "Migration notes" section of CHANGELOG.md + (create this section at the top of the Unreleased section if not already present) From a29442302f954f900527224725df7ecebce638ec Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 09:21:34 -0700 Subject: [PATCH 02/10] visibility --- docs/style-guide.md | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/docs/style-guide.md b/docs/style-guide.md index 8bf101218..b5d8a20da 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -59,6 +59,11 @@ methods. ## Java Language Conventions +### Visibility modifiers + +Follow the principle of minimal necessary visibility. Use the most restrictive access modifier that +still allows the code to function correctly. + ### Package conventions Classes in `.internal` packages are not considered public API and may change without notice. These @@ -117,6 +122,12 @@ Following the reasoning from Prefer AssertJ assertions over JUnit assertions (assertEquals, assertTrue, etc.) for better error messages. +### JUnit + +Test classes and test methods should generally be package-protected (no explicit visibility +modifier) rather than `public`. This follows the principle of minimal necessary visibility and is +sufficient for JUnit to discover and execute tests. + ### AutoService Use the `@AutoService` annotation when implementing SPI interfaces. This automatically generates the @@ -140,7 +151,7 @@ public class MyCustomizerProvider implements AutoConfigurationCustomizerProvider ## Configuration - Use `otel.` prefix for all configuration property keys -- Read config via the `ConfigProperties` interface +- Read configuration via the `ConfigProperties` interface - Provide sensible defaults and document all options - Validate configuration early with clear error messages From 066eb132f2033acc799f5f64e6c0bd93f0156592 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 09:57:02 -0700 Subject: [PATCH 03/10] Also add basic coding agent instructions --- .github/copilot-instructions.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index f5c985dd5..f7ac77115 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -28,3 +28,11 @@ This repository provides observability instrumentation for Java applications. - OpenTelemetry specification and semantic convention compliance - Resource cleanup and lifecycle management - Comprehensive unit tests for new functionality + +## Coding Agent Instructions + +When implementing changes or new features: + +1. Follow all [Style Guide](../docs/style-guide.md) conventions and the Code Review Priorities above +2. Run tests to ensure they still pass (use `./gradlew test` and `./gradlew integrationTest` as needed) +3. **Always run `./gradlew spotlessApply`** after making code changes to ensure proper formatting From b71cc4191076128ee5002cd6f66fc68ddd677143 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 11:06:59 -0700 Subject: [PATCH 04/10] clarifications --- docs/style-guide.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/style-guide.md b/docs/style-guide.md index b5d8a20da..33297c0af 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -33,7 +33,7 @@ Consider statically importing the following commonly used methods and constants: - `java.util.Arrays.*` (asList, etc.) - `java.util.Collections.*` (singletonList, emptyList, emptyMap, emptySet, etc.) - `java.util.stream.Collectors.*` (toList, toMap, toSet, etc.) - - `java.util.Objects` (requireNonNull, etc.) + - `java.util.Objects` (requireNonNull) - `java.util.logging.Level.*` (FINE, INFO, WARNING, etc.) - `java.nio.charset.StandardCharsets.*` (UTF_8, etc.) - **Time unit constants** @@ -75,9 +75,9 @@ packages contain implementation details that should not be used by external cons ### `final` keyword usage -Public API classes should be declared `final` where possible. +Public non-internal classes should be declared `final` where possible. -Methods should only be declared `final` if they are in non-final public API classes. +Methods should only be declared `final` if they are in public non-internal non-final classes. Fields should be declared `final` where possible. From 52463e1bf305fe3b52371948491d0fa9381ba531 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 11:37:20 -0700 Subject: [PATCH 05/10] update --- docs/style-guide.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/style-guide.md b/docs/style-guide.md index 33297c0af..76eb3a5be 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -30,16 +30,16 @@ Consider statically importing the following commonly used methods and constants: - `org.mockito.ArgumentMatchers.*` (any, eq, anyLong, etc.) - **Utility methods** - `io.opentelemetry.api.common.AttributeKey.*` (stringKey, longKey, etc.) - - `java.util.Arrays.*` (asList, etc.) + - `java.util.Arrays.*` (asList, stream, etc.) - `java.util.Collections.*` (singletonList, emptyList, emptyMap, emptySet, etc.) - `java.util.stream.Collectors.*` (toList, toMap, toSet, etc.) - - `java.util.Objects` (requireNonNull) + - `java.util.Objects.requireNonNull` - `java.util.logging.Level.*` (FINE, INFO, WARNING, etc.) - `java.nio.charset.StandardCharsets.*` (UTF_8, etc.) - **Time unit constants** - `java.util.concurrent.TimeUnit.*` (SECONDS, etc.) - **OpenTelemetry semantic convention constants** - - All constants under `io.opentelemetry.semconv.**` + - All constants under `io.opentelemetry.semconv.**`, except for `io.opentelemetry.semconv.SchemaUrls.*` constants. ### Class organization From 218f229ea26b72157da401bbed93f4577d8bb7b7 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 12:32:25 -0700 Subject: [PATCH 06/10] fix --- docs/style-guide.md | 8 -------- 1 file changed, 8 deletions(-) diff --git a/docs/style-guide.md b/docs/style-guide.md index 76eb3a5be..6adfac631 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -96,14 +96,6 @@ Annotate all parameters and fields that can be `null` with `@Nullable` (specific **Defensive programming**: Public APIs should still check for `null` parameters even if not annotated with `@Nullable`. Internal APIs do not need these checks. -**Enforcement**: Use the `otel.errorprone-conventions` Gradle plugin in all modules: - -```kotlin -plugins { - id("otel.errorprone-conventions") -} -``` - ### `Optional` usage Following the reasoning from From ae9711bb8cf3ad52f0c5477eee7623714cda4391 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 12:35:25 -0700 Subject: [PATCH 07/10] more --- docs/style-guide.md | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/docs/style-guide.md b/docs/style-guide.md index 6adfac631..faffa0f9c 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -39,7 +39,8 @@ Consider statically importing the following commonly used methods and constants: - **Time unit constants** - `java.util.concurrent.TimeUnit.*` (SECONDS, etc.) - **OpenTelemetry semantic convention constants** - - All constants under `io.opentelemetry.semconv.**`, except for `io.opentelemetry.semconv.SchemaUrls.*` constants. + - All constants under `io.opentelemetry.semconv.**`, except for + `io.opentelemetry.semconv.SchemaUrls.*` constants. ### Class organization @@ -64,7 +65,7 @@ methods. Follow the principle of minimal necessary visibility. Use the most restrictive access modifier that still allows the code to function correctly. -### Package conventions +### Internal packages Classes in `.internal` packages are not considered public API and may change without notice. These packages contain implementation details that should not be used by external consumers. @@ -75,7 +76,8 @@ packages contain implementation details that should not be used by external cons ### `final` keyword usage -Public non-internal classes should be declared `final` where possible. +Public non-internal classes should be declared `final` where possible. Internal and non-public +classes should not be declared `final`. Methods should only be declared `final` if they are in public non-internal non-final classes. From 3631faa8191c92ae4e523387ceca3acc137c011c Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 16:49:28 -0700 Subject: [PATCH 08/10] refine --- docs/style-guide.md | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/docs/style-guide.md b/docs/style-guide.md index faffa0f9c..91bad07e0 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -24,20 +24,21 @@ need to run `./gradlew spotlessApply` periodically. Consider statically importing the following commonly used methods and constants: - **Test methods** - - `io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions` (assertThat, satisfies, etc.) - - `org.assertj.core.api.Assertions.*` (assertThat, assertThatThrownBy, entry, etc.) - - `org.mockito.Mockito.*` (when, mock, verify, times, etc.) - - `org.mockito.ArgumentMatchers.*` (any, eq, anyLong, etc.) + - `io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.*` + - `org.assertj.core.api.Assertions.*` + - `org.mockito.Mockito.*` + - `org.mockito.ArgumentMatchers.*` - **Utility methods** - - `io.opentelemetry.api.common.AttributeKey.*` (stringKey, longKey, etc.) - - `java.util.Arrays.*` (asList, stream, etc.) - - `java.util.Collections.*` (singletonList, emptyList, emptyMap, emptySet, etc.) - - `java.util.stream.Collectors.*` (toList, toMap, toSet, etc.) - - `java.util.Objects.requireNonNull` - - `java.util.logging.Level.*` (FINE, INFO, WARNING, etc.) - - `java.nio.charset.StandardCharsets.*` (UTF_8, etc.) -- **Time unit constants** - - `java.util.concurrent.TimeUnit.*` (SECONDS, etc.) + - `io.opentelemetry.api.common.AttributeKey.*` + - `java.util.Arrays` - asList, stream + - `java.util.Collections` - singleton*, empty*, unmodifiable*, synchronized*, checked* + - `java.util.Objects` - requireNonNull + - `java.util.stream.Collectors.*` +- **Utility constants** + - `java.util.Locale.*` + - `java.util.concurrent.TimeUnit.*` + - `java.util.logging.Level.*` + - `java.nio.charset.StandardCharsets.*` - **OpenTelemetry semantic convention constants** - All constants under `io.opentelemetry.semconv.**`, except for `io.opentelemetry.semconv.SchemaUrls.*` constants. From d49f7c2f1303632aa08bb190db3706e45f5a9aa2 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Sun, 24 Aug 2025 20:25:42 -0700 Subject: [PATCH 09/10] up --- docs/style-guide.md | 1 + 1 file changed, 1 insertion(+) diff --git a/docs/style-guide.md b/docs/style-guide.md index 91bad07e0..54dc63cb5 100644 --- a/docs/style-guide.md +++ b/docs/style-guide.md @@ -33,6 +33,7 @@ Consider statically importing the following commonly used methods and constants: - `java.util.Arrays` - asList, stream - `java.util.Collections` - singleton*, empty*, unmodifiable*, synchronized*, checked* - `java.util.Objects` - requireNonNull + - `java.util.function.Function` - identity - `java.util.stream.Collectors.*` - **Utility constants** - `java.util.Locale.*` From 6e36706b5bd0cd2ac43e96e47417d800ab81e9ae Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Tue, 26 Aug 2025 07:54:09 -0700 Subject: [PATCH 10/10] Update .github/copilot-instructions.md --- .github/copilot-instructions.md | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index f7ac77115..9b11e2264 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -36,3 +36,4 @@ When implementing changes or new features: 1. Follow all [Style Guide](../docs/style-guide.md) conventions and the Code Review Priorities above 2. Run tests to ensure they still pass (use `./gradlew test` and `./gradlew integrationTest` as needed) 3. **Always run `./gradlew spotlessApply`** after making code changes to ensure proper formatting +4. Run markdown lint to ensure it still passes: `npx markdownlint-cli@0.45.0 -c .github/config/markdownlint.yml **/*.md`