search

StampyAI / stampede

(ALPHA) chat bot serving multiple services and configurations simultaneously
2 stars 0 forks source link

Use normal Elixir configuration instead of arguments #26

Closed ProducerMatt closed 1 month ago

ProducerMatt commented 1 month ago

Currently some configuration details are passed as arguments to the start callback. See:

        Stampede.Application.start(
          :normal,
          installed_services: [:dummy],
          services: [:dummy],
          log_to_file: false,
          log_post_serious_errors: false,
          clear_state: true
        )

It would be better to just have these args be set in an environment-specific config, like ./config/config_test.exs.

This also means disabling the manual startup and mix test --no-start and just being like every other app

ProducerMatt commented 1 month ago

Closed in https://github.com/StampyAI/stampede/commit/b71948d50fd5fb2378606db98e96181d153792fa