Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Suppression dependance stringr #140

Merged
merged 15 commits into from
Aug 23, 2023
Merged

feat: Suppression dependance stringr #140

merged 15 commits into from
Aug 23, 2023

Conversation

MurielleDelmotte
Copy link
Contributor

@MurielleDelmotte MurielleDelmotte commented Jul 18, 2023

pourquoi:

  • supprimer la dependance à stringr

quoi:

  • remplacement des fonctions de stringr par du rbase

Issue #51

  • Est-ce que mon code respecte les standards de qualité de mise en production de packages ?
  • Est-ce que la personne qui révise à toutes les informations pour valider les fonctionnalités / résolutions de problèmes sans trop de recherches ?
  • Est-ce que le client qui validera les tickets associés à les informations pour le faire sans perte de temps ?

Issues à faire valider pour fermer :

Issues traitées à garder ouvertes ou en cours :

  • issue #

Checklist:

  • Est-ce que le check du package passe en local ?
  • Est-ce que le CI passe ?
  • Est-ce que les fonctionnalités ajoutées / corrigées, sont documentées, testées ?
  • Est-ce que les fonctionnalités ajoutées / problèmes résolus sont brièvement présentées dans le message de la MR ?
  • Est-ce que les modifications sont liées à des tickets / issues que j'ai listés dans les commits et dans la MR elle-même ?
  • Est-ce que les tickets sont en mode "révision" dans le Board de suivi du projet ?
  • Est-ce que chaque ticket, s'il doit être fermé après acceptation de la MR contient un commentaire qui dit comment le valider ?

pourquoi:
- supprimer la dependance à stringr

quoi:
- remplacement des fonctions de stringr par du rbase

Issue #51
@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2023

Codecov Report

Merging #140 (4f282c6) into main (310678e) will decrease coverage by 0.13%.
The diff coverage is 42.85%.

❗ Current head 4f282c6 differs from pull request most recent head 8f67785. Consider uploading reports for the commit 8f67785 to get more accurate results

@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
- Coverage   78.78%   78.65%   -0.13%     
==========================================
  Files          63       63              
  Lines        1480     1471       -9     
==========================================
- Hits         1166     1157       -9     
  Misses        314      314              
Files Changed Coverage Δ
R/convert_file_to_dsfr.R 0.00% <0.00%> (ø)
R/convert_to_dsfr.R 0.00% <0.00%> (ø)
R/parse_html.R 100.00% <ø> (ø)
R/parse_to_xml.R 100.00% <ø> (ø)
R/update_inline.R 0.00% <ø> (ø)
R/get_dsfr_version.R 100.00% <100.00%> (ø)
R/render_node.R 95.65% <100.00%> (-1.23%) ⬇️

@jengelaere jengelaere merged commit 4010c30 into main Aug 23, 2023
@jengelaere jengelaere deleted the 51_remove_stringr branch August 23, 2023 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants