Почему .npmrc
ищут токен Github?
Кроме того, почему не проверяется process.env.NPM_TOKEN
, как GH_TOKEN
?
Может ли auto-release
внедрить $NPM_TOKEN
в .npmrc
для пользователя? (потенциально полезно в сборках CI)
Это корпоративная вещь GitHub. @adierkens может рассказать вам больше, поскольку я не сталкивался с этим.
Может ли автоматический выпуск внедрить $NPM_TOKEN в .npmrc для пользователя? (потенциально полезно в сборках CI)
Мне это нравится. это должно быть выполнимо
Эта функция, вероятно, должна быть включена только в CI. Если бы он запускался локально, вы могли бы случайно зафиксировать токен.
Будет ли этот пакет выполнять желаемое поведение, которое вы ищете?
https://www.npmjs.com/package/@hutson/set-npm-auth-token-for-ci
Конечно, но просмотр исходного кода показывает очень простую реализацию, поэтому было бы лучше включить ее в auto-release
, чем заставлять пользователя устанавливать другую зависимость.
Кажется, что он делает то, что делает довольно хорошо:
Это набор функций, которые, если бы мы хотели охватить все варианты использования, нам также пришлось бы кодировать все вышеперечисленное (или, по крайней мере, использовать те же пакеты).
В некоторых других случаях мы делаем то же самое. Мы полагаемся на parse-author , чтобы правильно проанализировать автора git, и на parse-github-url для URL-адреса github. Так что было бы несложно использовать для этого пакет.
Мне также нравится зависеть от таких пакетов, чтобы в мире было меньше дублирования кода. Любой, используя этот пакет и обнаруживая проблемы с ним, мы также можем решить проблемы для будущих разработчиков!
Моя основная проблема с пакетом заключается в том, как они также поставляют CLI с API. Для этого я открыл тему здесь
А пока нам следует раскошелиться на пакет (и, может быть, даже открыть PR). 😁
Да, я планирую сделать PR! Просто хочу знать, что хотел бы автор. Но лучший путь для этого пакета, вероятно, состоит в том, чтобы сделать и то, и другое: создать монорепозиторий и использовать меньшую библиотеку CLI.
Сопровождающий уже реализовал это! https://gitlab.com/hyper-expanse/open-source/set-npm-auth-token-for-ci/issues/2