Skip to content
New issue

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

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

Already on GitHub? # to your account

save only the TLM filename instead of the whole path #85

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

jmbhughes
Copy link
Member

Solves the confusion between whether a file exists.

Copy link

codecov bot commented Nov 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.60%. Comparing base (67ef92e) to head (36a021c).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #85   +/-   ##
=======================================
  Coverage   89.60%   89.60%           
=======================================
  Files          16       16           
  Lines         760      760           
=======================================
  Hits          681      681           
  Misses         79       79           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jmbhughes jmbhughes merged commit d0200c8 into main Nov 27, 2024
5 of 7 checks passed
@jmbhughes jmbhughes deleted the save-tlm-filename-only branch November 27, 2024 22:54
@jmbhughes jmbhughes self-assigned this Dec 4, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant