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

[sql]: Make config accessible for sql package #423

Closed
wants to merge 3 commits into from

Conversation

dlen
Copy link
Contributor

@dlen dlen commented Sep 24, 2024

Minimal change to be able to configure sql (mysql at least) storages with new_with_config()

@geofmureithi
Copy link
Owner

geofmureithi commented Sep 24, 2024

Config is already public.

pub struct Config {

just do use apalis_sql::Config

@dlen
Copy link
Contributor Author

dlen commented Sep 24, 2024

I tried that already. I have this on my cargo.toml

apalis = { version = "0.5", features = ["mysql"] }

That doesn't export apalis_sql only apalis::mysql and that doesn't export Config. Should I also add apalis-sql package??

@geofmureithi
Copy link
Owner

At the end of the day apalis_sql is already included by the mysql feature. Just use it also with the mysql feature.
As of version 0.6 you will have to do this.

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.

2 participants