restify/node-restify

maintenance branch releases accidentally blow away "latest" npm dist-tag

Open

#1.392 aberto em 29 de jun. de 2017

Ver no GitHub
 (8 comments) (0 reactions) (1 assignee)JavaScript (10.695 stars) (983 forks)batch import
CriticalFeatureHelp Wanted

Description

Currently on the 4.x branch (now a maintenance branch since 5.0.0 has been released), the make cutarelease target is used to tag and publish a new release. npm publish however will implicitly assign the 'latest' dist tag to this releae. We don't want that for 4.x, because 5.x releases should stay as the "latest".

Proposal: Use npm publish --tag=latest-4. Explicitly using a tag gets npm publish to not implicitly use 'latest'. I'm copying npm's own "latest-$majorver" pattern here:

$ npm info npm dist-tags

{ latest: '5.0.4',
  next: '5.0.4',
  'latest-2': '2.15.12',
  'next-2': '2.15.12',
  'latest-1': '1.4.29',
  lts: '2.15.11',
  'latest-3': '3.10.10',
  'next-3': '3.10.10',
  'latest-4': '4.6.1',
  'next-4': '4.6.1',
  'latest-5': '5.0.4',
  'next-5': '5.0.4',
  'v5.0-next': '5.0.4' }

@retrohacker Sound good?

Guia do colaborador

Pilha de tecnologia
nodejsshell
Domain
releasedevops
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
npm dist tagsrelease process
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
The issue proposes modifying the `make cutarelease` target in the Makefile to use `npm publish tag=latest 4` instead of just `npm publish` to avoid overwriting the latest dist tag on the 4.x maintenance branch. Investigate the current Makefile and cutarelease script to locate the publish command. Confirm that using ` tag=latest 4` will work as intended by checking npm documentation or the referenced npm repo pattern. Ensure that the fix does not affect other branches.