Reuse `Replacement` in `Regex.replace`
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 30/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Domaine
- api
Piste de recherche
Start by inspecting the Regex.replace API and the existing Replacement type mentioned in the issue. Determine whether replacing the String argument is appropriate and how the breaking change or deprecated alternative should be handled; done requires a decided API path and corresponding implementation and compatibility tests.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Right now the signature of replace on regex is:
foreign import replace :: Regex -> String -> String -> String
and I argue it should be
foreign import replace :: Regex -> Replacement -> String -> String
Of course this would be breaking so potentially there should be an alternative under a different name and the old one could be made deprecated.
What do you think?
- Langage dominant
- PureScript
- Étoiles
- 57
- Forks
- 76
- 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 purescript/purescript-strings
-
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
purescript/purescript-strings#176 · 6 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
purescript/purescript-strings#171 · 3 commentaires · 1 réaction ·
-
Add takeEnd & dropEndOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
purescript/purescript-strings#170 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
purescript/purescript-strings#169 · 7 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
purescript/purescript-strings#167 · 4 commentaires ·
Toutes les issues de purescript/purescript-strings
Issues similaires
-
ToTriage
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
microsoftgraph/msgraph-metadata#1217 ·
Les mainteneurs répondent en général sous 4 jours
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
jaegertracing/jaeger-ui#4547 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 1 jour
-
area:dictation bug P1 regression
Difficulté 2/5 1-3 heures Accessibilité débutants 87/100
uttrflow/uttrflow-swift#3387 ·
Les mainteneurs répondent en général sous 1 jour
-
area:cpan-port area:database bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
fglock/PerlOnJava#1604 ·
Les mainteneurs répondent en général sous 1 jour