Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Provide list of reloaded files #2307

Merged
merged 4 commits into from Nov 16, 2021
Merged

Provide list of reloaded files #2307

merged 4 commits into from Nov 16, 2021

Conversation

ahopkins
Copy link
Member

@ahopkins ahopkins commented Nov 9, 2021

This is a simple change to the reloader that provides insight into the files that were recently touched when a reload occurs. This change is meant to go along with some of the more recent changes to the auto-reloader (see #2167) that make the auto-reloader more usable.

@ahopkins ahopkins requested a review from a team as a code owner November 9, 2021 08:31
@codeclimate
Copy link

codeclimate bot commented Nov 15, 2021

Code Climate has analyzed commit e19f34c and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (86% is the threshold).

This pull request will bring the total coverage in the repository to 86.6% (0.0% change).

View more on Code Climate.

@ahopkins ahopkins added the needs review the PR appears ready but requires a review label Nov 15, 2021
@ahopkins ahopkins changed the title Allow access to reloaded files Provide list of reloaded files Nov 15, 2021
@ahopkins ahopkins merged commit abeb8d0 into main Nov 16, 2021
@ahopkins ahopkins deleted the reloaded-files branch November 16, 2021 08:16
@ahopkins ahopkins removed the needs review the PR appears ready but requires a review label Nov 16, 2021
ChihweiLHBird pushed a commit to ChihweiLHBird/sanic that referenced this pull request Jun 1, 2022
* Allow access to reloaded files

* Return to simple boolean values

* Resolve before adding to changed files
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.

None yet

2 participants