Skip to content

Enhancement: Resolve values for keys referring to ::class pseudo constants from the original name instead of the fully-qualified class name#265

Merged
localheinz merged 1 commit intomainfrom
feature/original-name
Dec 21, 2025
Merged

Enhancement: Resolve values for keys referring to ::class pseudo constants from the original name instead of the fully-qualified class name#265
localheinz merged 1 commit intomainfrom
feature/original-name

Conversation

@localheinz
Copy link
Member

This pull request

  • resolves values for keys referring to ::class pseudo constants from the original name instead of the fully-qualified class name

@codecov
Copy link

codecov bot commented Dec 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.14%. Comparing base (5469ab4) to head (4599c2f).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #265      +/-   ##
============================================
+ Coverage     59.86%   61.14%   +1.27%     
- Complexity       34       36       +2     
============================================
  Files             4        4              
  Lines           152      157       +5     
============================================
+ Hits             91       96       +5     
  Misses           61       61              

☔ 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.

@localheinz localheinz force-pushed the feature/original-name branch from 9ba7718 to 54d5d81 Compare December 21, 2025 09:34
Comment on lines 135 to +136
Bicycle::class => 'bicycle',
Namespaced\Apple::class => 'apple',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sorting order feels more natural.

@localheinz localheinz force-pushed the feature/original-name branch 2 times, most recently from 7332879 to 719a7bf Compare December 21, 2025 09:43
…tants from the original name instead of the fully-qualified class name
@localheinz localheinz force-pushed the feature/original-name branch from 719a7bf to 4599c2f Compare December 21, 2025 09:45
@localheinz localheinz merged commit 7de4b72 into main Dec 21, 2025
26 checks passed
@localheinz localheinz deleted the feature/original-name branch December 21, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant