Skip to content

tools/mpremote: Add tell() support for remote mounted files. - #19663

Open
dpgeorge wants to merge 1 commit into
micropython:masterfrom
dpgeorge:tools-mpremote-add-remote-file-tell
Open

tools/mpremote: Add tell() support for remote mounted files.#19663
dpgeorge wants to merge 1 commit into
micropython:masterfrom
dpgeorge:tools-mpremote-add-remote-file-tell

Conversation

@dpgeorge

@dpgeorge dpgeorge commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

This allows .tell() to be executed on remote mounted files, reusing the existing .seek() for simple implementation.

This is intended to replace #18298.

Testing

Added a test, it passes on PYBV10.

Trade-offs and Alternatives

#18298 implements this in a different, more complex way.

Generative AI

I did not use generative AI tools when creating this PR.

@dpgeorge
tools/mpremote: Add tell() support for remote mounted files.
97e17ac 
This allows `.tell()` to be executed on remote mounted files, reusing the
existing `.seek()` for simple implementation.

Signed-off-by: Damien George <damien@micropython.org>
@dpgeorge dpgeorge added the tools Relates to tools/ directory in source, or other tooling label Aug 28, 2026
@dpgeorge dpgeorge mentioned this pull request Aug 28, 2026
Closed
@codecov

codecov Bot commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.55%. Comparing base (117e9fa) to head (97e17ac).

Additional details and impacted files 
@@            Coverage Diff             @@
##           master   #19663      +/-   ##
==========================================
- Coverage   98.58%   98.55%   -0.04%     
==========================================
  Files         182      182              
  Lines       23322    23322              
  Branches        5        5              
==========================================
- Hits        22993    22985       -8     
- Misses        328      336       +8     
  Partials        1        1
Flag Coverage Δ
unix-coverage-32bit 98.55% <ø> (-0.04%) ⬇️
unix-coverage-64bit 98.52% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

Code size report:

Reference:  tools/mpremote: Ensure test_eval_exec_run has access to /tmp/run.py. [117e9fa]
Comparison: tools/mpremote: Add tell() support for remote mounted files. [merge of 97e17ac]
  mpy-cross:    +0 +0.000% 
   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
      esp32:    +0 +0.000% ESP32_GENERIC
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS
  qemu rv32:    +0 +0.000% VIRT_RV32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

tools Relates to tools/ directory in source, or other tooling

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@dpgeorge