Refactoring the index.js file of AWS and GCP plugin by renaming the parameter
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- aws, gcp, javascript
- Domaine
- cloud
Piste de recherche
Commencez par packages/common/core/base-provider.js à la ligne 58, puis comparez les déclarations des paramètres dans packages/aws-plugin/index.js et packages/gcp-plugin/index.js. Vérifiez que l’argument de configuration est nommé de manière cohérente dans les deux plugins et que la refactorisation ne modifie pas leur comportement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
We are passing in the configuration details to the plugin from the ncConfig file via the common package
https://github.com/leopardslab/nodecloud/blob/c8597192259598cbb90261b75ccb8aaf209deb78/packages/common/core/base-provider.js#L58
here the parameter for taking the configuration details from the ncConfig file is named options.
https://github.com/leopardslab/nodecloud/blob/c8597192259598cbb90261b75ccb8aaf209deb78/packages/aws-plugin/index.js#L4
Similarly, the GCP plugin is also taking in the configuration details as a parameter named options.
https://github.com/leopardslab/nodecloud/blob/c8597192259598cbb90261b75ccb8aaf209deb78/packages/gcp-plugin/index.js#L13
It would be easy to understand if we rename the options as configuration details
- Langage dominant
- JavaScript
- Étoiles
- 95
- Forks
- 87
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de leopardslab/nodecloud
-
Bugs in Readme.mdPeut-être pris @hs05june l’a pris il y a 1338 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 62/100
leopardslab/nodecloud#149 ·
-
Write unit tests for Oracle generator modulePeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
leopardslab/nodecloud#179 ·
-
Write unit tests for Linode generator modulePeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
leopardslab/nodecloud#178 ·
-
Implement Transformer for LinodePeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 20/100
leopardslab/nodecloud#169 ·
-
Implement Transformer for Oracle cloud providerPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
leopardslab/nodecloud#168 ·
Toutes les issues de leopardslab/nodecloud
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
AOSSIE-Org/BabyNest#284 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
seek4science/seek#2762 ·
Les mainteneurs répondent en général sous 5 jours
-
feedback simulation workshop
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
githubnext/gh-aw-workshop#5009 ·
Les mainteneurs répondent en général sous 1 jour
-
Clinic sharing: fast sync refusal message suggests Delete Data, which only developer builds haveOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
avniproject/avni-client#2228 ·
-
Add: K+ Sport 2 LogoOuvertecheck:passed logos:add
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 6 jours