Any notes at all are considered breaking
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- release
Piste de recherche
Commencez par la condition liée dans lib/analyze-commit.js, puis comparez la manière dont les notes sont produites dans writer-opts.js de conventional-changelog-angular. Reproduisez l’analyse de la release pour un commit squashé contenant des notes ordinaires et l’exemple de feat imbriqué ; le travail est terminé lorsque les notes ordinaires ne provoquent plus de release breaking, tandis que les véritables notes breaking continuent de le faire.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
We maintain a custom conventional changelog link that we would like to use with semantic-release, but any time we've tried this, any time we merge a PR it's treated as a breaking change because of this line right here:
(!breaking || (commit.notes && commit.notes.length > 0)) &&
It treats any commit note at all as a breaking change. I'm pretty sure this is because the angular conventional discards all notes unless they are a breaking change https://github.com/conventional-changelog/conventional-changelog/blob/master/packages/conventional-changelog-angular/writer-opts.js#L31-L34 (I checked a few other conventional changelogs and did not see them using the notes field at all).
This breaks our conventional changelog because we maintain detailed notes for each of the commits inside of a squashed commit. We use this in other places to analyze the squashed commits to determine what the correct release type is. For example, if a commit looks like this:
fix: Something small
* feat: Add a feat
Then the correct release for that squashed commit would be a feat. Semantic release has this concept itself when multiple commits are released at once, we do the same thing just at a PR level.
That problematic line above means that any PR that is squashed is treated as a breaking change though, which means we end up having to maintain a separate set of release rules which defeats the whole purpose of creating a conventional-changelog for our org.
IMO that line should be changed to scan for BREAKING CHANGE in the notes instead of default treating the presence of a note s a breaking change.
- Langage dominant
- JavaScript
- Étoiles
- 447
- Forks
- 82
- Merge moyen
- 7 h 53 min
- PR mergées (30 j)
- 9
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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.
Autres issues de semantic-release/commit-analyzer
-
released on @beta
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
semantic-release/commit-analyzer#921 · 8 commentaires · 7 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
semantic-release/commit-analyzer#805 · 6 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
semantic-release/commit-analyzer#802 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Unexpected major release because commit description includes a line starting with "Breaking change"Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
semantic-release/commit-analyzer#776 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Exclamation mark (!) after type/scope completely ignores commitPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertereleased on @beta
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
semantic-release/commit-analyzer#759 · 13 commentaires · 6 réactions ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de semantic-release/commit-analyzer
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
nodejs/node#66743 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
OWASP/cve-lite-cli#1335 ·
Les mainteneurs répondent en général sous 1 jour
-
Channels status: needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
mastra-ai/mastra#27093 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
category/development priority/P1 scope/ci-cd scope/testing status/ready-for-agent type/bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
QwenLM/qwen-code#14225 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100