Skip to content

Conversation

@hongsu
Copy link
Contributor

@hongsu hongsu commented Aug 21, 2025

As requested by @aylei in #6798, this PR adds helm lint to the helm unittest workflow.

See review comment: #6798 (review)

Tested (run the relevant ones):

  • Code formatting: install pre-commit (auto-check on commit) or bash format.sh
  • Any manual or new tests for this PR (please specify below)
  • All smoke tests: /smoke-test (CI) or pytest tests/test_smoke.py (local)
  • Relevant individual tests: /smoke-test -k test_name (CI) or pytest tests/test_smoke.py::test_name (local)
  • Backward compatibility: /quicktest-core (CI) or pytest tests/smoke_tests/test_backward_compat.py (local)

Signed-off-by: Hongsu Ryu <hongsu.ryu@moreh.io>
Copy link
Collaborator

@aylei aylei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @hongsu-moreh !

@aylei aylei enabled auto-merge (squash) August 21, 2025 09:20
@aylei aylei merged commit 7747439 into skypilot-org:master Aug 21, 2025
16 checks passed
massaindustries pushed a commit to Seeweb/skypilot that referenced this pull request Aug 26, 2025
add to helm lint for github's' workflow

Signed-off-by: Hongsu Ryu <hongsu.ryu@moreh.io>
@seongsukwon-moreh seongsukwon-moreh deleted the modify-helm-ci-workflow branch September 11, 2025 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants