Skip to content

add syntax checks and tests to language workflows#2

Merged
krrrr38 merged 1 commit into
mainfrom
improve-ci
Jun 5, 2026
Merged

add syntax checks and tests to language workflows#2
krrrr38 merged 1 commit into
mainfrom
improve-ci

Conversation

@krrrr38
Copy link
Copy Markdown
Member

@krrrr38 krrrr38 commented Jun 5, 2026

No description provided.

Signed-off-by: krrrr38 <k.kaizu38@gmail.com>
@github-actions github-actions Bot added the ci label Jun 5, 2026
with:
php-version: '8.5'
- run: composer install --no-interaction --prefer-dist
- run: find src -name "*.php" | xargs -I{} php -l {}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2038:warning:1:1: Use 'find .. -print0 | xargs -0 ..' or 'find .. -exec .. +' to allow non-alphanumeric filenames [shellcheck]

cache: pip
- run: pip install .
- run: pip install ".[dev]"
- run: python -m py_compile $(find src -name "*.py")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2046:warning:1:22: Quote this to prevent word splitting [shellcheck]

ruby-version: '3.3'
bundler-cache: true
working-directory: ruby
- run: find lib -name "*.rb" | xargs -I{} ruby -c {}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2038:warning:1:1: Use 'find .. -print0 | xargs -0 ..' or 'find .. -exec .. +' to allow non-alphanumeric filenames [shellcheck]

@krrrr38 krrrr38 merged commit 7ae387a into main Jun 5, 2026
11 of 12 checks passed
@krrrr38 krrrr38 deleted the improve-ci branch June 5, 2026 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant