Any notes at all are considered breaking
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- release
Línea de trabajo
Empieza por la condición enlazada en lib/analyze-commit.js y compara después cómo se generan las notas en writer-opts.js de conventional-changelog-angular. Reproduce el análisis de la release para un commit fusionado mediante squash que contenga notas ordinarias y el ejemplo de feat anidado; se considera terminado cuando las notas ordinarias ya no provocan una release breaking, pero las notas breaking genuinas sí lo siguen haciendo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- JavaScript
- Estrellas
- 447
- Forks
- 82
- Merge medio
- 7 h 53 min
- PR fusionados (30 d)
- 9
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de semantic-release/commit-analyzer
-
released on @beta
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
semantic-release/commit-analyzer#921 · 8 comentarios · 7 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
semantic-release/commit-analyzer#805 · 6 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
semantic-release/commit-analyzer#802 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Unexpected major release because commit description includes a line starting with "Breaking change"Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
semantic-release/commit-analyzer#776 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Exclamation mark (!) after type/scope completely ignores commitPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoreleased on @beta
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
semantic-release/commit-analyzer#759 · 13 comentarios · 6 reacciones ·
Los mantenedores suelen responder en 1 día
Todos los issues de semantic-release/commit-analyzer
Issues similares
-
Language MVP task
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
saayam-for-all/webapp#1914 ·
Los mantenedores suelen responder en 1 día
-
[Bug] Dev image build fails: `npm install` in the root Dockerfile runs out of memory (exit 134)Posiblemente ocupada @andest01 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
good first issue hacktoberfest help wanted
Dificultad 2/5 Menos de una hora Aptitud para principiantes 76/100
pacocartones/free-llm-api-hub#262 ·
Los mantenedores suelen responder en 1 día
-
agent-task risk:low
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
guyellis/http-status-check#512 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
microsoft/playwright#43327 ·
Los mantenedores suelen responder en 1 día