Skip to content

👷 Add GitHub workflow for linting and code validation#5

Merged
rickyheijnen merged 1 commit intomainfrom
config/add-code-validation-workflow
Sep 27, 2025
Merged

👷 Add GitHub workflow for linting and code validation#5
rickyheijnen merged 1 commit intomainfrom
config/add-code-validation-workflow

Conversation

@rickyheijnen
Copy link
Copy Markdown
Member

@rickyheijnen rickyheijnen commented Sep 27, 2025

🔍 Summary

This PR adds comprehensive GitHub Actions workflows for automated code quality checks including linting, static analysis, and validation across multiple file types and PHP versions.

📝 Description

  • PHP CS Fixer integration with caching and artifact upload for failed runs
  • PHPStan analysis across PHP 7.4-8.3 matrix and Psalm integration with conditional execution

💬 Additional information

The workflows are designed to run on every push and provide comprehensive code quality checks. The static analysis runs across multiple PHP versions to ensure compatibility, while the linting workflows validate various file formats commonly used in PHP projects.

@rickyheijnen rickyheijnen self-assigned this Sep 27, 2025
coderabbitai[bot]

This comment was marked as resolved.

@rickyheijnen rickyheijnen force-pushed the config/add-code-validation-workflow branch 6 times, most recently from d0d6cf6 to c2838c4 Compare September 27, 2025 21:16
@rickyheijnen rickyheijnen force-pushed the config/add-code-validation-workflow branch from c2838c4 to b6ca698 Compare September 27, 2025 21:20
@brixion brixion deleted a comment from coderabbitai bot Sep 27, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 27, 2025

Warning

Rate limit exceeded

@rickyheijnen has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 30 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between efc3e23 and cd5587d.

📒 Files selected for processing (2)
  • .github/workflows/code-lint.yml (1 hunks)
  • .github/workflows/code-validate.yml (1 hunks)

\n\n## Walkthrough\nAdds two GitHub Actions workflows under .github/workflows: one to run PHP CS Fixer in dry-run mode on push with caching and failure artifacts, and another to run PHPStan static analysis across PHP 7.4–8.3 with caching, conditional config selection, and artifact upload on failure.\n\n## Changes\n| Cohort / File(s) | Summary of changes |\n| --- | --- |\n| CI workflows
\\.github/workflows/code-lint.yml, \\.github/workflows/code-validate.yml | Introduces Code Lint workflow (PHP 8.2, composer caching, PHP CS Fixer dry-run, cache artifact on failure) and Code Validate workflow (PHPStan matrix on PHP 7.4–8.3, composer and PHPStan cache, conditional config/paths, artifact upload on failure). |\n\n## Estimated code review effort\n🎯 2 (Simple) | ⏱️ ~10 minutes\n\n

\n\n## Pre-merge checks and finishing touches\n
\n✅ Passed checks (3 passed)\n\n| Check name | Status | Explanation |\n| :----------------: | :------- | :--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |\n| Title Check | ✅ Passed | The title succinctly describes the main change of adding GitHub Actions workflows for linting and code validation, directly matching the addition of the code-lint and code-validate pipelines in the diff. It is clear and specific, though it includes an emoji which is non-essential. |\n| Description Check | ✅ Passed | The description clearly outlines the addition of comprehensive GitHub Actions workflows for linting and static analysis across multiple PHP versions, which aligns with the PR’s implementation of PHP CS Fixer and PHPStan jobs. It directly relates to the changeset and provides relevant context for the workflows being introduced. |\n| Docstring Coverage | ✅ Passed | No functions found in the changes. Docstring coverage check skipped. |\n\n
\n\n

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]

This comment was marked as resolved.

@rickyheijnen rickyheijnen force-pushed the config/add-code-validation-workflow branch from b6ca698 to ab650da Compare September 27, 2025 21:34
@rickyheijnen rickyheijnen force-pushed the config/add-code-validation-workflow branch from ab650da to 84a0bd5 Compare September 27, 2025 21:39
@rickyheijnen rickyheijnen force-pushed the config/add-code-validation-workflow branch 3 times, most recently from 457d0cd to 2995d9a Compare September 27, 2025 22:09
@rickyheijnen rickyheijnen force-pushed the config/add-code-validation-workflow branch from 2995d9a to cd5587d Compare September 27, 2025 22:11
@rickyheijnen rickyheijnen merged commit 0eb01a9 into main Sep 27, 2025
7 checks passed
@rickyheijnen rickyheijnen deleted the config/add-code-validation-workflow branch September 27, 2025 22:12
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