Skip to content

TST: Replace tmpdir with tmp_path and minor clean-up#258

Merged
lpsinger merged 2 commits intoastropy:mainfrom
pllim:replace-tmpdir
Mar 6, 2025
Merged

TST: Replace tmpdir with tmp_path and minor clean-up#258
lpsinger merged 2 commits intoastropy:mainfrom
pllim:replace-tmpdir

Conversation

@pllim
Copy link
Member

@pllim pllim commented Mar 6, 2025

Saw a failure about this in astropy/astropy#17861

@codecov
Copy link

codecov bot commented Mar 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.23%. Comparing base (46857f5) to head (67ac6b1).
Report is 11 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #258   +/-   ##
=======================================
  Coverage   96.23%   96.23%           
=======================================
  Files           6        6           
  Lines         505      505           
=======================================
  Hits          486      486           
  Misses         19       19           

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

@pllim pllim marked this pull request as ready for review March 6, 2025 20:53
@pllim pllim requested a review from lpsinger March 6, 2025 20:53
@lpsinger lpsinger merged commit c8bbbfd into astropy:main Mar 6, 2025
12 checks passed
@pllim pllim deleted the replace-tmpdir branch March 6, 2025 21:06
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.

2 participants