Skip to content

Commit

Permalink
Fix type annotation List -> list (#27044)
Browse files Browse the repository at this point in the history
#26934 added the old style
`List` which is not imported anymore and tests are currently broken.

### Testing
The `helm` suite in buildkite tests should pass.
  • Loading branch information
shalabhc authored Jan 11, 2025
1 parent a712a8e commit 32dc531
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions helm/dagster/schema/schema/charts/dagster/subschema/daemon.py
Original file line number Diff line number Diff line change
Expand Up @@ -107,5 +107,5 @@ class Daemon(BaseModel, extra="forbid"):
volumeMounts: Optional[list[kubernetes.VolumeMount]] = None
volumes: Optional[list[kubernetes.Volume]] = None
initContainerResources: Optional[kubernetes.Resources] = None
extraContainers: Optional[List[kubernetes.Container]] = None
extraPrependedInitContainers: Optional[List[kubernetes.InitContainer]] = None
extraContainers: Optional[list[kubernetes.Container]] = None
extraPrependedInitContainers: Optional[list[kubernetes.InitContainer]] = None
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from typing import List, Optional
from typing import Optional

from pydantic import BaseModel

Expand All @@ -7,5 +7,5 @@

class Migrate(BaseModel):
enabled: bool
extraContainers: Optional[List[kubernetes.Container]]
initContainers: Optional[List[kubernetes.Container]]
extraContainers: Optional[list[kubernetes.Container]]
initContainers: Optional[list[kubernetes.Container]]
Original file line number Diff line number Diff line change
Expand Up @@ -48,5 +48,5 @@ class Webserver(BaseModel, extra="forbid"):
volumeMounts: Optional[list[kubernetes.VolumeMount]] = None
volumes: Optional[list[kubernetes.Volume]] = None
initContainerResources: Optional[kubernetes.Resources] = None
extraContainers: Optional[List[kubernetes.Container]] = None
extraPrependedInitContainers: Optional[List[kubernetes.InitContainer]] = None
extraContainers: Optional[list[kubernetes.Container]] = None
extraPrependedInitContainers: Optional[list[kubernetes.InitContainer]] = None

0 comments on commit 32dc531

Please sign in to comment.