Skip to content

Do relinking during rimport#20

Open
samsrabin wants to merge 10 commits intoESMCI:mainfrom
samsrabin:relink-in-rimport
Open

Do relinking during rimport#20
samsrabin wants to merge 10 commits intoESMCI:mainfrom
samsrabin:relink-in-rimport

Conversation

@samsrabin
Copy link
Contributor

This PR makes it so that rimport does the relinking step too. If relinking is all done successfully (or no new files were published), a message is printed saying that there's no need to run relink.py.

Resolves #14.

@samsrabin samsrabin requested a review from jedwards4b February 7, 2026 00:12
@samsrabin samsrabin self-assigned this Feb 7, 2026
@samsrabin samsrabin added the enhancement New feature or request label Feb 7, 2026
@codecov-commenter
Copy link

codecov-commenter commented Feb 7, 2026

Codecov Report

❌ Patch coverage is 99.40476% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 99.13%. Comparing base (ff55606) to head (2010909).
⚠️ Report is 49 commits behind head on main.

Files with missing lines Patch % Lines
tests/rimport/test_check_relink_worked.py 96.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
+ Coverage   97.76%   99.13%   +1.37%     
==========================================
  Files          21       28       +7     
  Lines        2010     2899     +889     
==========================================
+ Hits         1965     2874     +909     
+ Misses         45       25      -20     

☔ 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

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Combine rimport and relink

2 participants