Modernize CI with Dependabot, GitHub Actions, and Maven wrapper#105
Merged
mercyblitz merged 5 commits intomicrosphere-projects:dev-1.xfrom Mar 27, 2026
Merged
Modernize CI with Dependabot, GitHub Actions, and Maven wrapper#105mercyblitz merged 5 commits intomicrosphere-projects:dev-1.xfrom
mercyblitz merged 5 commits intomicrosphere-projects:dev-1.xfrom
Conversation
Add Dependabot config and modernize CI/build tooling: update GitHub Actions to use checkout@v5 and setup-java@v5, switch build steps to use ./mvnw, and enhance maven-publish with input validation, a release job (tag/release creation and next-patch bump). Add mvnw and mvnw.cmd, upgrade maven wrapper to 3.3.4 and point distribution to an Aliyun mirror. Also apply widespread JavaDoc/comment improvements and minor pom adjustments across service registry and discovery modules, and remove redundant test logback resources.
Ensure the Maven wrapper is executable before invoking it in the GitHub Actions job to avoid permission errors on runners where the execute bit may not be preserved. Adds `chmod +x mvnw` prior to running `./mvnw --batch-mode --update-snapshots --file pom.xml`.
Add a dedicated step to chmod +x mvnw before the build and remove the redundant chmod from the 'Build with Maven' step. This ensures the Maven wrapper is executable on the runner and keeps the build step focused on running ./mvnw with the existing flags.
Delete the unit test file FeignClientSpecificationPostProcessorTest from microsphere-spring-cloud-openfeign. The removed test covered injection of AUTO_REFRESH_CAPABILITY_CLASS into FeignClientSpecification; the file has been removed from the test suite (deleted at src/test/java/io/microsphere/spring/cloud/openfeign/autoconfigure/FeignClientSpecificationPostProcessorTest.java).
Refactor: pull the trace logging into a static helper method log(Class<?>[], Class<?>[]) and invoke it from the post-processor. This makes the logging logic reusable and easier to test. Add FeignClientSpecificationPostProcessorTest that calls the new log method (with LoggingLevelsTest configured) to ensure the logging path is exercised. No functional behavior change.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request makes several improvements to the project's build and release automation, updates documentation and example usage in core classes, and upgrades Maven-related configuration. The key changes are grouped below.
Build and Release Automation Improvements:
.github/dependabot.ymlconfiguration to enable daily Maven dependency update checks via Dependabot.actions/checkout@v5,actions/setup-java@v5), switched Maven build to use the Maven Wrapper (./mvnw), and added a step to ensure the wrapper script is executable.Maven Wrapper and Repository Updates:
maven-wrapper.propertiesto use Maven 3.9.9, set the wrapper version to 3.3.4, changed the distribution type toonly-script, and switched the Maven distribution URL to use Aliyun's mirror for improved reliability in some regions.Documentation and Example Usage Improvements:
ReactiveDiscoveryClientAdapter,UnionDiscoveryClient,DiscoveryClientAutoConfiguration,ReactiveDiscoveryClientAutoConfiguration,InMemoryServiceRegistry, andMultipleAutoServiceRegistrationto make the APIs easier to understand and use. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]Dependency and Metadata Clarification:
microsphere-spring-cloud-commons/pom.xmlto indicate that the dependency is for testing purposes.