From 4e424ddee68c8a0b173b9f98d9ed4378047842dd Mon Sep 17 00:00:00 2001 From: Seyit Zor Date: Tue, 21 Nov 2023 10:31:15 +0100 Subject: [PATCH] Delete pylint ignore lines Co-authored-by: Byron Himes --- src/metldata/artifacts_rest/config.py | 1 - src/metldata/artifacts_rest/models.py | 1 - src/metldata/builtin_transformations/aggregate/config.py | 1 - src/metldata/builtin_transformations/merge_slots/config.py | 1 - src/metldata/event_handling/artifact_events.py | 1 - src/metldata/load/collect.py | 1 - src/metldata/submission_registry/models.py | 1 - 7 files changed, 7 deletions(-) diff --git a/src/metldata/artifacts_rest/config.py b/src/metldata/artifacts_rest/config.py index 6b85f7c1..814c000e 100644 --- a/src/metldata/artifacts_rest/config.py +++ b/src/metldata/artifacts_rest/config.py @@ -30,7 +30,6 @@ class ArtifactsRestConfig(BaseSettings): description="Information for artifacts to be queryable via the Artifacts REST API.", ) - # pylint: disable=no-self-argument @field_validator("artifact_infos") def validate_artifact_info_names( cls, value: list[ArtifactInfo] diff --git a/src/metldata/artifacts_rest/models.py b/src/metldata/artifacts_rest/models.py index 66b7c156..d82eeec3 100644 --- a/src/metldata/artifacts_rest/models.py +++ b/src/metldata/artifacts_rest/models.py @@ -74,7 +74,6 @@ class ArtifactInfo(BaseModel): ), ) - # pylint: disable=no-self-argument @field_validator("resource_classes") def check_resource_class_names( cls, value: dict[str, ArtifactResourceClass] diff --git a/src/metldata/builtin_transformations/aggregate/config.py b/src/metldata/builtin_transformations/aggregate/config.py index 431268c5..e25d6335 100644 --- a/src/metldata/builtin_transformations/aggregate/config.py +++ b/src/metldata/builtin_transformations/aggregate/config.py @@ -37,7 +37,6 @@ class AggregationOperation(BaseModel): visit_only_once: Optional[list[str]] = None function: type[AggregationFunction] - # pylint: disable=no-self-argument @model_validator(mode="before") def lookup_operation(cls, values: dict) -> dict: """Replaces operation strings with operation types.""" diff --git a/src/metldata/builtin_transformations/merge_slots/config.py b/src/metldata/builtin_transformations/merge_slots/config.py index b54eaebd..93c2b81e 100644 --- a/src/metldata/builtin_transformations/merge_slots/config.py +++ b/src/metldata/builtin_transformations/merge_slots/config.py @@ -46,7 +46,6 @@ class SlotMergingConfig(BaseSettings): ], ) - # pylint: disable=no-self-argument @field_validator("merge_instructions") def validate_merge_instructions( cls, filtered_merge_instructions: list[SlotMergeInstruction] diff --git a/src/metldata/event_handling/artifact_events.py b/src/metldata/event_handling/artifact_events.py index 0559ebf2..422f5f3f 100644 --- a/src/metldata/event_handling/artifact_events.py +++ b/src/metldata/event_handling/artifact_events.py @@ -31,7 +31,6 @@ class ArtifactEventConfig(BaseModel): ), ) - # pylint: disable=no-self-argument @field_validator("artifact_topic_prefix") def artifact_topic_prefix_must_not_contain_dots(cls, value: str): """Validate that artifact topic prefix does not contain dots.""" diff --git a/src/metldata/load/collect.py b/src/metldata/load/collect.py index 29d9cfb3..3b5c5abc 100644 --- a/src/metldata/load/collect.py +++ b/src/metldata/load/collect.py @@ -40,7 +40,6 @@ class ArtifactCollectorConfig(ArtifactEventConfig): ), ) - # pylint: disable=no-self-argument @field_validator("artifact_types") def artifact_types_must_not_contain_dots(cls, value: list[str]): """Validate that artifact types do not contain dots.""" diff --git a/src/metldata/submission_registry/models.py b/src/metldata/submission_registry/models.py index 962a9a55..65322d61 100644 --- a/src/metldata/submission_registry/models.py +++ b/src/metldata/submission_registry/models.py @@ -105,7 +105,6 @@ def current_status(self) -> SubmissionStatus: return sorted_history[-1].new_status - # pylint: disable=no-self-argument @field_validator("accession_map") def check_accession_uniqueness(cls, value: AccessionMap) -> AccessionMap: """Check that no accessions are re-used accross classes."""