NativeScript / NativeScript/nativescript-cli
option to set location for sourcemaps (for console logs)
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 1.1k
- Forks
- 204
- Merge moyen
- 1 j 9 h
- PR mergées (30 j)
- 8
Description
Ok so to get sourcemaps fully working with Sentry, i have to modify quite a few things:
- change the location of sourcemaps (like with hidden sourcemaps)
- force
sourceMappingURLto follow the formatapp:///bundle.js.map(so no mention to the relative path where source maps are located).
Consequently sourceMapConverter will fail here because this is the wrong location. I guess it tries to find the sourcemap in the same dir as the source itself. Or even what does it do with app:///
Do you think it would be possible to define somewhere (project wise) the location of the sourcemaps. May be using https://github.com/thlorenz/convert-source-map#frommapfilecommentcomment-mapfiledir ?
Guide de contribution
Ouvrir le 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.
Piste de recherche
Commencez par lire lib/services/log-source-map-service.ts au niveau de l'appel à sourceMapConverter et suivez la manière dont les emplacements de sourcemap de console-log sont résolus. Vérifiez la référence à convert-source-map dans l'issue ainsi que les chemins de configuration existants du projet. C'est terminé lorsqu'un emplacement de sourcemap au niveau du projet peut prendre en charge le format Sentry demandé app:///bundle.js.map sans interrompre la recherche des sources.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- cli
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100