Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

ci: add K&R code style checker workflow#3005

Open
kwizer15 wants to merge1 commit intojeedom:alpha
base:alpha
Choose a base branch
Loading
fromkwizer15:feat/knr-style-check

Conversation

kwizer15
Copy link
Contributor

Ajout d'un vérificateur de style K&R

Description

Cette PR ajoute un système de vérification automatique du style de code K&R avec une approche non disruptive :

Principales fonctionnalités

  • Workflow GitHub Actions qui vérifie uniquement les lignes modifiées lors des PR
  • Script de vérification utilisable en local
  • Documentation multilingue (FR/EN/DE/ES)
  • Possibilité de correction automatique via phpcbf
  • Support pour git pre-commit hook

Composants ajoutés

  • .github/workflows/phpcs.yml : Configuration du workflow GitHub
  • .github/scripts/check-style.sh : Script de vérification
  • .github/phpcs/kr.xml : Règles de style K&R
  • docs/{lang}/code-styling.md : Documentation dans différentes langues

Impact sur le code existant

  • Aucune modification du code existant
  • Les vérifications ne s'appliquent qu'aux nouvelles modifications
  • Installation et utilisation optionnelles pour les développeurs

Tests effectués lors du développement de la PR

  • Vérification locale sur différentes modifications
  • Test du workflow GitHub sur des PR de test
  • Validation des règles K&R sur le code nouveau vs existant

Notes

Le système respecte le style historique du projet tout en fournissant des outils pour maintenir la cohérence sur les nouvelles contributions.

Suggested changelog entry

Added K&R style checking system

Related issues/external references

N/A

Types of changes

  • Bug fix(non-breaking change which fixes)
  • New feature(non-breaking change which adds functionality)
  • Breaking change(fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

- Add GitHub Actions workflow for PR style validation- Setup K&R style ruleset based on original Jeedom conventions- Check only modified PHP files in PR- Provide feedback as PR comments- Keep the process external to project dependencies
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

7 participants
@kwizer15@zoic21@Hotfirenet@tmartinez69009@Salvialf@Sekiro-kost@reineabs

[8]ページ先頭

©2009-2025 Movatter.jp