Refacto/#380 extract licenses apis to support manifestmanager input #384
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ager dans extract.ts - Ajout de deux fonctions, `getManifestManagerAndDirAsync` et `getManifestManagerAndDirSync`, pour gérer les instances de ManifestManager et les chemins de répertoire. - Modification des fonctions `extractLicenses` et `extractLicensesSync` pour utiliser ces nouvelles fonctions. - Mise à jour des tests pour vérifier le bon fonctionnement avec des instances de ManifestManager.
…t getManifestManagerAndDirSync - Suppression de paramètres inutiles dans les fonctions asynchrones et synchrones pour une meilleure clarté. - Amélioration de la gestion des erreurs en centralisant les exceptions pour les types d'entrée non valides. - Mise à jour de l'appel à getManifestManagerAndDirAsync dans la fonction extractLicenses.
|
|
Je viens de me rendre compte que j'ai tout fais dans la même PR... dsl 😞 |
|
@davy-dev Ce serait bien de faire une PR à part pour gérer la location sur mama (tu pourras toujours rebase celle-ci). |
…synchrone - Changement de la déclaration de la fonction getManifestManagerAndDirAsync pour la rendre asynchrone, permettant une meilleure gestion des opérations asynchrones.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Voici la PR pour l'issue 380 j'èspère avoir bien compris ce qui était attendu 😅
Au départ j'avais implémenté directement la logic de gestion des entrées dans extractLicenses et extractLicensesSynch puis je me dis on ne sais jamais ... comme je découvre le projet j'ai créé dans le même fichier des utilitaires pour séparer un peu les choses. j'espère que c'est pertinent par rapport à l'utilisation du scanner. Cette semaine je vais faire de la lecture pour apprendre plus sur SPDX ✊
Refactor: Supporte l’entrée ManifestManager dans extractLicenses & extractLicensesSync
Changements principaux
manifestLocationà la classeManifestManager(dans le workspacemama), pour mémoriser le chemin du manifeste lors de l’instanciation.extractLicensesetextractLicensesSync(dans le workspaceconformance) :ManifestManager.Checklist
manifestLocationàManifestManager