Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correction de la dépendance InfluxDB dans composer.json pour compatibilité avec le code actuel #3038

Open
wants to merge 1 commit into
base: alpha
Choose a base branch
from

Conversation

kwizer15
Copy link
Contributor

@kwizer15 kwizer15 commented Feb 22, 2025

Description

Cette PR corrige une dépendance inadaptée dans le fichier composer.json. Initialement, la dépendance "influxdata/influxdb-client-php": "^3" était utilisée, mais elle n’est pas compatible avec le code actuel de Jeedom qui repose sur une implémentation plus ancienne de la bibliothèque InfluxDB. J’ai donc remplacé cette dépendance par "influxdb/influxdb-php": "^1.15", qui correspond à la version attendue par le code existant.

J’ai également mis à jour le fichier composer.lock pour refléter ce changement. Cependant, comme ce fichier est supprimé lors de l’installation avant de lancer composer install, cette mise à jour n’a pas d’impact direct pour les utilisateurs finaux ou les développeurs.

Le choix de la version ^1.15 a été fait pour garantir une stabilité avec la dernière version mineure disponible dans la série 1.x tout en restant compatible avec le code actuel.

Corrige les erreurs PHPStan suivantes : https://github.com/jeedom/core/pull/3033/files?short_path=57271cb#diff-995edee38ad4f8387e58ebd52c31bcc04c56cc2448d331b1cf5e0b35c57b9efaR183-R194

Suggested changelog entry

  • Correction de la dépendance InfluxDB dans composer.json pour restaurer la compatibilité avec le code actuel.

Related issues/external references

Issue #3039

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

  • I have checked there is no other PR open for the same change.
  • I have read the [La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project).
  • I grant the project the right to include and distribute the code under the GNU.
  • I have added tests to cover my changes. (Note : si vous n’avez pas ajouté de tests, laissez cette case vide et expliquez pourquoi si pertinent dans la description)
  • I have verified that the code complies with the projects coding standards.
  • [Required for new sniffs] I have added MD documentation for the sniff. (Non applicable ici)

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