Skip to content

Commit

Permalink
Merge pull request #1125 from neXenio/chore/fix-linter
Browse files Browse the repository at this point in the history
chore: fix linter
  • Loading branch information
ademariag authored Jan 26, 2024
2 parents 9002f1d + 0124d63 commit 5a7b4f8
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/test-build-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,6 @@ jobs:
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v4
with:
python-version: '3.10'
- uses: psf/black@main

test:
Expand Down
6 changes: 3 additions & 3 deletions kapitan/inputs/helm.py
Original file line number Diff line number Diff line change
Expand Up @@ -257,9 +257,9 @@ class HelmChart(BaseModel):

def new(self):
for obj in self.load_chart():
self.root[
f"{obj['metadata']['name'].lower()}-{obj['kind'].lower().replace(':','-')}"
] = BaseObj.from_dict(obj)
self.root[f"{obj['metadata']['name'].lower()}-{obj['kind'].lower().replace(':','-')}"] = (
BaseObj.from_dict(obj)
)

def load_chart(self):
helm_values_file = None
Expand Down
5 changes: 3 additions & 2 deletions tests/test_compile.py
Original file line number Diff line number Diff line change
Expand Up @@ -152,8 +152,9 @@ def test_compile_not_enough_args(self):
self.assertEqual(cm.exception.code, 1)

def test_compile_not_matching_targets(self):
with self.assertLogs(logger="kapitan.targets", level="ERROR") as cm, contextlib.redirect_stdout(
io.StringIO()
with (
self.assertLogs(logger="kapitan.targets", level="ERROR") as cm,
contextlib.redirect_stdout(io.StringIO()),
):
# as of now, we cannot capture stdout with contextlib.redirect_stdout
# since we only do logger.error(e) in targets.py before exiting
Expand Down

0 comments on commit 5a7b4f8

Please sign in to comment.