Skip to content

chore: standardize lint workflows#243

Merged
philippemnoel merged 4 commits intomainfrom
chore/standardize-lint-workflows
Mar 6, 2026
Merged

chore: standardize lint workflows#243
philippemnoel merged 4 commits intomainfrom
chore/standardize-lint-workflows

Conversation

@philippemnoel
Copy link
Member

Summary

  • Remove unused "Install fd Search Tool" step from lint-format.yml (fd-find was installed but never used)
  • Remove redundant if: github.event.pull_request.draft == false conditions
  • Use uv tool run instead of pip install for beautysh in lint-bash.yml
  • Accept both #!/bin/bash and #!/usr/bin/env bash shebangs
  • Accept both set -euo pipefail and set -Eeuo pipefail with optional leading whitespace
  • Use improved ShellCheck runner with graceful no-files handling and -P scripts flag
  • Remove redundant --ignore-path .prettierignore from lint-yaml.yml (prettier reads it by default)
  • Standardize step names and comments across all lint workflows

Aligns with the lint workflows in django-paradedb.

Test plan

  • Verify lint-format CI passes
  • Verify lint-bash CI passes
  • Verify lint-markdown CI passes
  • Verify lint-yaml CI passes
  • Verify lint-pr-title CI passes

- Remove unused "Install fd Search Tool" step from lint-format.yml
- Remove redundant if: draft == false conditions
- Use uv tool run instead of pip install for beautysh in lint-bash.yml
- Accept both #!/bin/bash and #!/usr/bin/env bash shebangs
- Accept both set -euo and set -Eeuo pipefail with optional leading whitespace
- Use improved ShellCheck runner with graceful no-files handling
- Remove redundant --ignore-path .prettierignore from lint-yaml.yml
- Standardize step names and comments across all lint workflows
@vercel
Copy link

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
website Ready Ready Preview, Comment Mar 6, 2026 1:23am

Request Review

@philippemnoel philippemnoel merged commit 49e3ea0 into main Mar 6, 2026
8 of 9 checks passed
@philippemnoel philippemnoel deleted the chore/standardize-lint-workflows branch March 6, 2026 01:21
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.

1 participant