Skip to content

ci: sdk-5413 validate workflows and package artifacts - #135

Open
dciccale wants to merge 1 commit into
masterfrom
ci/sdk-5413-pre-release-validation
Open

ci: sdk-5413 validate workflows and package artifacts#135
dciccale wants to merge 1 commit into
masterfrom
ci/sdk-5413-pre-release-validation

Conversation

@dciccale

@dciccale dciccale commented Sep 2, 2026

Copy link
Copy Markdown

Summary

  • build the source distribution and wheel during pull-request validation
  • run twine check against both package artifacts
  • run actionlint with embedded ShellCheck support across all workflows
  • run ShellCheck against repository-owned shell scripts
  • update the PR workflow checkout and Python setup actions from Node 20 to Node 24 versions

Safety

Validation

  • actionlint passed all workflows with ShellCheck integration
  • ShellCheck passed setup_env.sh
  • python -m build --no-isolation produced the wheel and source distribution
  • python -m twine check dist/* passed both artifacts
  • both updated JavaScript actions declare the Node 24 runtime
  • git diff --check passed

Linear: https://linear.app/rudderstack/issue/SDK-5413/add-python-sdk-pre-release-package-and-workflow-validation

@dciccale
dciccale requested a review from a team as a code owner September 2, 2026 16:23

- name: Checkout
uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4.3.0
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we use the latest version here?

Comment on lines +63 to +74
- name: Validate workflows and embedded shell
uses: reviewdog/action-actionlint@83e4ed25b168066ad8f62f5afbb29ebd8641d982 # v1.69.1
with:
reporter: local
filter_mode: nofilter
fail_level: any

- name: Validate repository shell scripts
shell: bash
run: >-
git ls-files -z -- '*.sh' |
xargs -0 --no-run-if-empty shellcheck --shell=bash

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are these checks only needed for this repository or can be extended for others as well?

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