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

FileWatcher: Don't type-check changes #56

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

leandro-lucarella-frequenz
Copy link
Contributor

This check is unnecessary, as the underlying library is typed. This is not really signaling the closing of the "channel".

Fixes #54.

@leandro-lucarella-frequenz leandro-lucarella-frequenz added this to the v0.11.0 milestone Nov 23, 2022
@leandro-lucarella-frequenz leandro-lucarella-frequenz added the type:tech-debt Improves the project without visible changes for users label Nov 23, 2022
@github-actions github-actions bot added the part:core Affects the core types (`Sender`, `Receiver`, exceptions, etc.) label Nov 23, 2022
This check is unnecessary, as the underlying library is typed. This is
not really signaling the closing of the "channel".

Fixes frequenz-floss#54.

Signed-off-by: Leandro Lucarella <[email protected]>
@leandro-lucarella-frequenz
Copy link
Contributor Author

Not sure what's wrong with GitHub that is not picking up the action results, I will force-rebase and force-push to make the action run.

@leandro-lucarella-frequenz
Copy link
Contributor Author

Force-merging too as I didn't change anything since your review.

@leandro-lucarella-frequenz leandro-lucarella-frequenz merged commit 2319161 into frequenz-floss:v0.x.x Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:core Affects the core types (`Sender`, `Receiver`, exceptions, etc.) type:tech-debt Improves the project without visible changes for users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FileWatcher shouldn't check if changes has the correct type
3 participants