Skip to content

Conversation

@SkyeBeFreeman
Copy link
Collaborator

@SkyeBeFreeman SkyeBeFreeman commented Nov 3, 2025

PR Type

Refactoring (no functional changes, no api changes).

Describe what this PR does for and how you did.

Adding the issue link (#xxx) if possible.

closes #1725

Note

Checklist

  • Add information of this PR to CHANGELOG.md in root of project.
  • Add documentation in javadoc or comment below the PR if necessary.

@SkyeBeFreeman SkyeBeFreeman added this to the 2.1.1.x milestone Nov 3, 2025
@SkyeBeFreeman SkyeBeFreeman changed the title refactor:optimize config locate. #1741 refactor:optimize config locate. Nov 3, 2025
@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

❌ Patch coverage is 84.12698% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.10%. Comparing base (7414328) to head (48c53b0).

Files with missing lines Patch % Lines
...ris/config/configdata/PolarisConfigDataLoader.java 79.31% 4 Missing and 2 partials ⚠️
...laris/config/adapter/PolarisConfigFileLocator.java 80.00% 1 Missing and 1 partial ⚠️
...olaris/config/adapter/PolarisConfigFilePuller.java 91.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               2021    #1742      +/-   ##
============================================
+ Coverage     53.02%   53.10%   +0.08%     
  Complexity       63       63              
============================================
  Files           453      453              
  Lines         13046    13001      -45     
  Branches       1606     1585      -21     
============================================
- Hits           6917     6904      -13     
+ Misses         5572     5548      -24     
+ Partials        557      549       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

合并ConfigDataLoader和PropertySourceLocator的动态配置文件实现

1 participant