Skip to content

Comments

fix(hotreload): do not watch the entire directory, only config file#2289

Merged
bnjjj merged 1 commit intodevfrom
bnjjj/fix_hot_reload
Dec 19, 2022
Merged

fix(hotreload): do not watch the entire directory, only config file#2289
bnjjj merged 1 commit intodevfrom
bnjjj/fix_hot_reload

Conversation

@bnjjj
Copy link
Contributor

@bnjjj bnjjj commented Dec 19, 2022

close #2287

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
@bnjjj bnjjj requested review from Geal and o0Ignition0o December 19, 2022 11:01
@bnjjj bnjjj self-assigned this Dec 19, 2022
@github-actions

This comment was marked as off-topic.

@bnjjj bnjjj enabled auto-merge (squash) December 19, 2022 11:02
Copy link
Contributor

@o0Ignition0o o0Ignition0o left a comment

Choose a reason for hiding this comment

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

The configuration supports a rhai scripts directory https://www.apollographql.com/docs/router/customizations/rhai/

is this being watched?

@bnjjj
Copy link
Contributor Author

bnjjj commented Dec 19, 2022

Not related to the hot reload of rhai

@bnjjj bnjjj requested a review from o0Ignition0o December 19, 2022 11:04
@bnjjj bnjjj disabled auto-merge December 19, 2022 11:08
@bnjjj bnjjj enabled auto-merge (squash) December 19, 2022 11:10
@bnjjj bnjjj merged commit 4633b37 into dev Dec 19, 2022
@bnjjj bnjjj deleted the bnjjj/fix_hot_reload branch December 19, 2022 11:18
@bnjjj bnjjj mentioned this pull request Dec 19, 2022
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.

hot reload regression

3 participants