search

torrust / torrust-tracker

A modern and feature-rich (private) BitTorrent tracker.
https://torrust.com
GNU Affero General Public License v3.0
357 stars 40 forks source link

Config overhaul: lowercase for `DatabaseDriver` #950

Closed josecelano closed 3 months ago

josecelano commented 3 months ago

Parent issue: https://github.com/torrust/torrust-tracker/issues/401

We are normalizing all enum variants to lowercase. DatabaseDriver is not normalized yet.

pub enum DatabaseDriver {
    /// The Sqlite3 database driver.
    Sqlite3,
    /// The `MySQL` database driver.
    MySQL,
}
josecelano commented 3 months ago

Hi @da2ce7 I think we should have two types for this concept. The enum was added to the primitives crate because it's also used in the configuration. However, I think the configuration and the database domain should be decoupled. I would add a new enum in the configuration crate and I would move this to the database mod.