plotly / plotly/plotly.R

Shouldn't s2 be as Suggested package?

Ouverte
#2,203 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
R
Étoiles
2.7k
Forks
641
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

It's required to run some tests, e.g.

https://github.com/plotly/plotly.R/blob/f9d6e43ff42760e84113992eb9711bc4fcbcf203/tests/testthat/test-ggplot-sf.R#L4

By the reasoning in WRE, that makes s2 Suggests:

All packages that are needed to successfully run R CMD check on the package must be listed in one of ‘Depends’ or ‘Suggests’ or ‘Imports’. Packages used to run examples or tests conditionally... should be listed in ‘Suggests’ or ‘Enhances’.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par tests/testthat/test-ggplot-sf.R à la ligne 4 et examinez les métadonnées des dépendances du package, puis comparez l’exigence aux recommandations de WRE concernant les dépendances de packages. Mettez à jour la classification si cela est approprié et exécutez R CMD check pour vérifier que les tests et les déclarations de dépendances fonctionnent correctement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
r
Domaine
build-system, testing-qa
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
50/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.