microsoft / microsoft/TypeScript-TmLanguage
"Install dependencies" needs additional command line options
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 471
- Forks
- 149
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
TS and JS Grammar Extension version: 0.0.50 / latest
In README.md, it is documented that to install dependencies, npm install is all that is required. However, I've found that on my Windows 10 machine with Visual Studio 2017, I have to type npm install -g windows-build-tools in a PowerShell running with administrator privileges or oniguruma fails to be installed.
It appears that after I get it to build one time, I don't have to pass those extra arguments to npm.
I found my solution when someone else had a similar problem on superuser.com. I'm logging this here to help others having the same problem, and as a suggestion that the README.md be updated to include the extra arguments needed on Windows 10.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Actualiza las instrucciones de instalación de dependencias de README.md para el caso reportado en Windows 10. Empieza revisando las indicaciones existentes sobre npm install y el fallo de oniguruma descrito en el issue; se considera terminado cuando el README documente claramente el comando adicional de PowerShell con privilegios de administrador y cuándo es necesario.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- documentation
- Tipo de issue
- Documentación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 55/100