how about removing function last_plot?
Nessuno ha ancora preso questa issue.
- Lingua principale
- R
- Stelle
- 2.7k
- Fork
- 641
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
plotly depends on ggplot2 which alreadly has loaded function last_plot(), so, maybe we can throw it away as ggridge did
Brief description of the problem
library(plotly)
#> Loading required package: ggplot2
#>
#> Attaching package: 'plotly'
#> The following object is masked from 'package:ggplot2':
#>
#> last_plot
#> The following object is masked from 'package:stats':
#>
#> filter
#> The following object is masked from 'package:graphics':
#>
#> layout
plotly::last_plot
#> function ()
#> .store$get()
#> <bytecode: 0x7f9575ad7558>
#> <environment: namespace:plotly>
ggplot2::last_plot
#> function ()
#> .store$get()
#> <bytecode: 0x7f9575ad5cd8>
#> <environment: namespace:ggplot2>
Created on 2021-04-19 by the reprex package (v2.0.0)
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia confrontando gli entry point pubblici plotly::last_plot e ggplot2::last_plot mostrati nel report, quindi esamina come plotly espone o testa last_plot. Determina il comportamento esistente del package e la copertura dei test prima di rimuovere il duplicato; il lavoro è completato quando plotly non maschera più last_plot di ggplot2 senza interrompere l’uso supportato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- r
- Ambito
- data-visualization
- Tipo di issue
- Refactoring
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100