Skip to content

Conversation

@Caelion
Copy link
Contributor

@Caelion Caelion commented Nov 10, 2025

Add functionality to list plugin package requirements regarding pip2 and pip3 to avoid check it manually in all plugin

Description

Add functionality to list plugin package requirements regarding pip2 and pip3 to avoid check it manually in all plugin

Suggested changelog entry

Page de mise à jour : Ajout d'une synthèse des paquets pip2 et pip3 utilisés par les plugins et leurs versions attendues afin de limiter les mises à jours dépassant les versions requises par les plugins.

Related issues/external references

Fixes #

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 functionality to list plugin package requirements regarding pip2 and pip3 to avoid check it manually in all plugin
@Mips2648
Copy link
Collaborator

  • comment in french to be deleted
  • this feature already exists in config > last tab > check system package
  • and anyway, this is useless under debian 12 and following because each plugin dependencies are installed separately so you will never have conflicts between plugins
  • it is not mandatory to have a package.json to manage dependencies so putting alert box in such case will create a lot of noise

@Mips2648 Mips2648 self-requested a review December 14, 2025 07:34
</div>

<?php
// --- Recherche de tous les packages.json dans les plugins ---
Copy link
Collaborator

Choose a reason for hiding this comment

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

comment in french

</table>
<?php } else { ?>
<div class="alert alert-info">
{{Aucun fichier packages.json trouvé dans les plugins.}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wouldn't write such alert

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.

2 participants