syn2mas: replace #[allow] annotations with #[expect]

Also removes unused #[allow] annotations.
This commit is contained in:
Quentin Gliech committed 2025-04-18 11:34:27 +02:00
1 parent cdb35d97ef
commit 59bcf1b963
3 files changed
+4 -7

No files matched your search

+1 -4
View File
@@ -46,7 +46,7 @@ pub enum Error {
},
#[error("writer connection pool shut down due to error")]
#[allow(clippy::enum_variant_names)]
#[expect(clippy::enum_variant_names)]
WriterConnectionPoolError,
#[error("inconsistent database: {0}")]
@@ -390,7 +390,6 @@ impl MasWriter {
/// Errors are returned in the following conditions:
///
/// - If the database connection experiences an error.
#[allow(clippy::missing_panics_doc)] // not real
#[tracing::instrument(name = "syn2mas.mas_writer.new", skip_all)]
pub async fn new(
mut conn: LockedMasDatabase,
@@ -632,7 +631,6 @@ impl MasWriter {
/// Errors are returned in the following conditions:
///
/// - If the database writer connection pool had an error.
#[allow(clippy::missing_panics_doc)] // not a real panic
#[tracing::instrument(skip_all, level = Level::DEBUG)]
pub fn write_users(&mut self, users: Vec<MasNewUser>) -> BoxFuture<'_, Result<(), Error>> {
self.writer_pool
@@ -711,7 +709,6 @@ impl MasWriter {
/// Errors are returned in the following conditions:
///
/// - If the database writer connection pool had an error.
#[allow(clippy::missing_panics_doc)] // not a real panic
#[tracing::instrument(skip_all, level = Level::DEBUG)]
pub fn write_passwords(
&mut self,
+1 -1
View File
@@ -140,7 +140,7 @@ struct MigrationState {
///
/// - An underlying database access error, either to MAS or to Synapse.
/// - Invalid data in the Synapse database.
#[allow(clippy::implicit_hasher)]
#[expect(clippy::implicit_hasher)]
pub async fn migrate(
mut synapse: SynapseReader<'_>,
mas: MasWriter,
+2 -2
View File
@@ -16,7 +16,7 @@ use sqlx::postgres::PgConnectOptions;
///
/// See: <https://element-hq.github.io/synapse/latest/usage/configuration/config_documentation.html>
#[derive(Deserialize)]
#[allow(clippy::struct_excessive_bools)]
#[expect(clippy::struct_excessive_bools)]
pub struct Config {
pub database: DatabaseSection,
@@ -239,7 +239,7 @@ mod test {
#[test]
fn test_to_sqlx_postgres() {
#[track_caller]
#[allow(clippy::needless_pass_by_value)]
#[expect(clippy::needless_pass_by_value)]
fn assert_eq_options(config: DatabaseSection, uri: &str) {
let config_connect_options = config
.to_sqlx_postgres()