vercel/release

Can't release if there's only 1 commit since last tag

Open

#93 ouverte le 30 juin 2017

Voir sur GitHub
 (3 commentaires) (2 réactions) (0 assignés)JavaScript (3 534 stars) (145 forks)batch import
bughelp wanted

Description

I end up with the error Error! No changes happened since the last release. if I have only 1 commit since the last tag.

Seems like the following code might be the issue https://github.com/zeit/release/blob/master/lib/commits.js#L22

  if (commit.hash === release.hash) {
    return
  }

Guide contributeur

Stack technique
javascriptnodejs
Domaine
toolingrelease
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsbasic git knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
The issue is in lib/commits.js at line 22 where the code returns early if the commit hash matches the release hash. This causes a failure when there is only one commit since the last tag. To fix, the logic should be adjusted to handle the single commit case, possibly by not returning or by checking the number of commits. There are no linked PRs, so the solution is open for implementation. The maintainer has not commented on the issue, so the scope is defined by the reported bug.