Skip to content
This repository has been archived by the owner on Oct 19, 2023. It is now read-only.

fix: remove some deprecated stuff #242

Open
wants to merge 1 commit into
base: mistress
Choose a base branch
from

Conversation

cyyynthia
Copy link
Contributor

I hate it when software yells at me with yellow stuff >:D

MessageDispatcher.send_message and Dispatch.send_to_clients no longer take the sender as an arg as it's gone unused; removed redundant signature for queries that cannot be routed. (it seems it was a hacky thing anyways, as it was nil more often than a socket 馃檮)

hopefully I didn't screw up this time and the build will work ugh

also remove some unused args and hopefully make build working
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

** (CompileError) config/config.exs:55: undefined function config_env/0
** (CompileError) config/config.exs:55: undefined function config_env/0
    (stdlib) lists.erl:1354: :lists.mapfoldl/3
    (elixir) expanding macro: Kernel.if/2
    config/config.exs:55: (file)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant