Skip to content

Conversation

@cadenmyers13
Copy link
Contributor

@cadenmyers13 cadenmyers13 commented Jan 27, 2026

closes #204

please see this closed PR for information on whats changed in this PR: #207. This PR was duplicated because of merge conflicts

@cadenmyers13
Copy link
Contributor Author

@sbillinge ready for review/merge. I created a new PR to clean the commit history and I also was struggling with the merge conflict for some reason.

@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.37%. Comparing base (f1e9b63) to head (f1d3930).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
+ Coverage   99.31%   99.37%   +0.06%     
==========================================
  Files           5        5              
  Lines         292      322      +30     
==========================================
+ Hits          290      320      +30     
  Misses          2        2              
Files with missing lines Coverage Δ
tests/test_tools.py 99.01% <100.00%> (+0.17%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cadenmyers13
Copy link
Contributor Author

@sbillinge can we get this merged?

@sbillinge sbillinge merged commit eee3608 into diffpy:main Jan 30, 2026
4 checks passed
@cadenmyers13 cadenmyers13 deleted the improved-cli branch January 30, 2026 14:27
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.

fix: update command-line interface to be more user-friendly

2 participants