[Amelioration] Remplacer la dépendance à stringr par du rbase

[MurielleDelmotte]

ETQ utilisateurs, j'aimerais que mon package ne dépende pas de {stringr} car celui si nécessite l'installation de {stringi} qui met beaucoup de temps à compiler.

Critère d'acceptation

• le package stringr n'est plus en dépendance de shinygouv dans le fichier DESCRIPTION

Critère technique

• remplacer stringr par du rbase

[jengelaere]

Bonjour à tous,
Je me suis penchée sur la demande de MR suite au message de Murielle sur slack (que j'ai lu de travers...).
Les checks passent nickel, juste quand je me place dans l'app test.convert.dsfr et que je tente une conversion au dsfr avec la fonction, ça ne va pas au bout avec l'erreur suivante Error in : header must be the output of header_dsfr() voici mes sorties consoles :

> golem::document_and_reload()

ℹ Loading test.convert.dsfr
ℹ Loading test.convert.dsfr
Warning messages:
1: remplacement de l'importation précédente ‘shiny::dataTableOutput’ par ‘DT::dataTableOutput’ lors du chargement de ‘test.convert.dsfr’ 
2: remplacement de l'importation précédente ‘shiny::renderDataTable’ par ‘DT::renderDataTable’ lors du chargement de ‘test.convert.dsfr’ 
3: remplacement de l'importation précédente ‘shiny::dataTableOutput’ par ‘DT::dataTableOutput’ lors du chargement de ‘test.convert.dsfr’ 
4: remplacement de l'importation précédente ‘shiny::renderDataTable’ par ‘DT::renderDataTable’ lors du chargement de ‘test.convert.dsfr’ 
 

# Run the application
> run_app()
Le chargement a nécessité le package : shiny
Listening on http://127.0.0.1:4286
> library(shinygouv)
Attachement du package : ‘shinygouv’
L'objet suivant est masqué depuis ‘package:test.convert.dsfr’:

> convert_to_dsfr()
scan R/app_config.R
scan R/app_server.R
scan R/app_ui.R
Attention le R/app_ui.R contient un 'tabPanel': 
- Si celui ci était dans un navbarPage alors le remplacer par un navbarPanel_dsfr. 
- Si celui ci était dans un 'tabsetPanel', alors le remplacer par un 'tabPanel_dsfr'
Attention le R/app_ui.R contient un 'navbarPage()', la version dsfr nécessite un header, voir \?navbarPage_dsfr
scan R/data_filtre.R
scan R/mod_droite.R
scan R/mod_gauche.R
scan R/run_app.R
scan R/utils-pipe.R
scan R/zzzz.R
la dependance a shinygouv a ete ajoute dans app_ui.R
[1] "Conversion terminee"
golem::document_and_reload()
ℹ Loading test.convert.dsfr
Writing NAMESPACE
ℹ Loading test.convert.dsfr
Warning messages:
1: remplacement de l'importation précédente ‘shiny::dataTableOutput’ par ‘DT::dataTableOutput’ lors du chargement de ‘test.convert.dsfr’ 
2: remplacement de l'importation précédente ‘shiny::renderDataTable’ par ‘DT::renderDataTable’ lors du chargement de ‘test.convert.dsfr’ 
3: remplacement de l'importation précédente ‘shiny::dataTableOutput’ par ‘DT::dataTableOutput’ lors du chargement de ‘test.convert.dsfr’ 
4: remplacement de l'importation précédente ‘shiny::renderDataTable’ par ‘DT::renderDataTable’ lors du chargement de ‘test.convert.dsfr’ 

# Run the application
> run_app()
Listening on http://127.0.0.1:4286

Warning: Error in : header must be the output of header_dsfr()
  76: dots_list
  75: tagList
  74: add_dsfr_deps [C:/Users/juliette.engelaere/Documents/Travail/R_local/shinygouv/R/shinygouv-dependencies.R#91]
  69: ui [C:/Users/juliette.engelaere/Documents/Travail/R_local/shinygouv/inst/test.convert.dsfr/R/app_ui.R#9]
   3: runApp
   2: print.shiny.appobj
   1: <Anonymous>

[jengelaere]

Ah y est j'ai compris, pourquoi... Je mets un ticket pour qu'on utilise une app de test de la conversion qui marche