fix ReloadException when in REPL by dhalbert · Pull Request #6168 · adafruit/circuitpython · GitHub
Skip to content

fix ReloadException when in REPL #6168

Merged
dhalbert merged 1 commit into
adafruit:7.2.xfrom
dhalbert:ReloadException-in-repl
Mar 16, 2022
Merged

fix ReloadException when in REPL #6168
dhalbert merged 1 commit into
adafruit:7.2.xfrom
dhalbert:ReloadException-in-repl

Conversation

@dhalbert

Copy link
Copy Markdown
Collaborator

Do not raise ReloadException if in the REPL.
Also undo autoreload suspend before triggering autoreload. This is not imperative, but if autoreload_trigger() were to check the suspend state (it does not currently), it would have the wrong value. This was originally necessary when I was testing a different fix.

Please test! I tested on macOS and Linux with a Metro M4 and a Feather RP2040,
modifying a running code.py, copying a 40kB file while running, and also editing code.py and copying a file when in the REPL.

@dhalbert dhalbert requested a review from tannewt March 16, 2022 16:56

@tannewt tannewt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Works well on Linux and Mac with file transfers and code.py edits from Sublime Text.

@dhalbert

Copy link
Copy Markdown
Collaborator Author

@dhalbert dhalbert merged commit 66fa923 into adafruit:7.2.x Mar 16, 2022
@dhalbert dhalbert deleted the ReloadException-in-repl branch March 16, 2022 18:33
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