Skip to content

Handle simultaneous update and deprecate/restore - #79

Open
MJGaughran wants to merge 4 commits into
mainfrom
hla-1150-simultaneous-update-deprecate
Open

Handle simultaneous update and deprecate/restore#79
MJGaughran wants to merge 4 commits into
mainfrom
hla-1150-simultaneous-update-deprecate

Conversation

@MJGaughran

Copy link
Copy Markdown
Contributor

At the moment, if you update a Module's contents as well as deprecate or restore it, only the update takes effect on sync. This means that the compare command will fail the next time it is run, as the modulefile is not in the correct location.

This only needed a fix to validation as deploy was already capable of handling Modules that fell into more than one category.

When creating the tests, it was clear that some places used "active" and others "live" to refer to a non-deprecated module, so I have cleared this up.

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.48%. Comparing base (4f5ee44) to head (08fde82).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
- Coverage   99.48%   99.48%   -0.01%     
==========================================
  Files          27       27              
  Lines         979      978       -1     
==========================================
- Hits          974      973       -1     
  Misses          5        5              

☔ View full report in Codecov by Harness.
📢 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.

1 participant