Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Organize and add new omegaconf resolvers, add utility to avoid dupes #115

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Sep 23, 2025
Merged

Organize and add new omegaconf resolvers, add utility to avoid dupes #115

merged 4 commits into from
Sep 23, 2025

Conversation

@timkpaine
Copy link
Member

@timkpaine timkpaine commented Sep 17, 2025

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 17, 2025 •
edited
Loading

Test Results

421 tests  +17   419 ✅ +17   22s ⏱️ -1s
  1 suites ± 0     2 💤 ± 0 
  1 files   ± 0     0 ❌ ± 0 

Results for commit 2f16dfb. ± Comparison against base commit d1cef37.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Sep 17, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 96.68874% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.81%. Comparing base (d1cef37) to head (2f16dfb).
⚠️ Report is 33 commits behind head on main.

Files with missing lines Patch % Lines
ccflow/plugins/omegaconf_resolvers.py 90.38% 2 Missing and 3 partials ⚠️
Additional details and impacted files 
@@           Coverage Diff            @@
##             main     #115    +/-   ##
========================================
  Coverage   95.80%   95.81%            
========================================
  Files         118      118            
  Lines        6467     6601   +134     
  Branches      405      415    +10     
========================================
+ Hits         6196     6325   +129     
- Misses        172      174     +2     
- Partials       99      102     +3

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

feussy
feussy previously approved these changes Sep 17, 2025
View reviewed changes
ptomecek
ptomecek previously requested changes Sep 17, 2025
View reviewed changes
@timkpaine timkpaine requested a review from ptomecek September 23, 2025 19:22
@timkpaine timkpaine requested a review from feussy September 23, 2025 19:48
@timkpaine timkpaine merged commit ded35a6 into main Sep 23, 2025
6 checks passed
@timkpaine timkpaine deleted the tkp/rs branch September 23, 2025 21:14
@timkpaine timkpaine added the type: enhancement Minor improvements label Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ptomecek ptomecek ptomecek approved these changes

@hintse hintse Awaiting requested review from hintse hintse is a code owner

@feussy feussy Awaiting requested review from feussy feussy is a code owner

Assignees

No one assigned

Labels

type: enhancement Minor improvements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@timkpaine @feussy @ptomecek