Skip to content
New issue

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

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

Already on GitHub? # to your account

[Breaking] Add support for watching multiple paths #15

Merged
merged 1 commit into from
Jul 8, 2020

Conversation

florimondmanca
Copy link
Owner

Fixes #11

Migration steps:

hotreload = arel.HotReload("./path", on_reload=[...])

becomes:

hotreload = arel.HotReload(
    paths=[
        arel.Path("./path", on_reload=[...]),
    ],
)

@florimondmanca florimondmanca changed the title Add support for watching multiple paths [Breaking] Add support for watching multiple paths Jul 8, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jul 8, 2020

Codecov Report

Merging #15 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9        11    +2     
  Lines          174       189   +15     
=========================================
+ Hits           174       189   +15     
Impacted Files Coverage Δ
src/arel/__init__.py 100.00% <100.00%> (ø)
src/arel/_app.py 100.00% <100.00%> (ø)
src/arel/_models.py 100.00% <100.00%> (ø)
src/arel/_types.py 100.00% <100.00%> (ø)
tests/test_example.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d41b58b...337c5a4. Read the comment docs.

@florimondmanca florimondmanca merged commit 4316c58 into master Jul 8, 2020
@florimondmanca florimondmanca deleted the multi-paths branch July 8, 2020 14:18
@florimondmanca florimondmanca mentioned this pull request Jul 8, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not able to register multiple hot reload scripts
2 participants