Skip to content

Conversation

@fsoubelet
Copy link
Member

Closes #30. Coincidentally, increases test coverage 😅

@fsoubelet fsoubelet requested a review from JoschD October 29, 2025 09:35
@fsoubelet fsoubelet self-assigned this Oct 29, 2025
@fsoubelet fsoubelet added Estimate: Easy Good first issue for newcomers. Straightforward fixes. Type: Maintenance Improvements in the code, that are not necessarily visible in functionality. Type: Release Issue preparing for a release. Status: Review Needed Work currently stopped, untils someone else reviews it. Priority: Medium Work on this. labels Oct 29, 2025
@github-actions
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  turn_by_turn
  __init__.py
  io.py
Project Total  

This report was generated by python-coverage-comment-action

@fsoubelet fsoubelet requested a review from JoschD October 29, 2025 10:55
@fsoubelet fsoubelet merged commit e8d4705 into master Oct 29, 2025
19 checks passed
@fsoubelet fsoubelet deleted the remove_esrf branch October 29, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Estimate: Easy Good first issue for newcomers. Straightforward fixes. Priority: Medium Work on this. Status: Review Needed Work currently stopped, untils someone else reviews it. Type: Maintenance Improvements in the code, that are not necessarily visible in functionality. Type: Release Issue preparing for a release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove ESRF reader

3 participants