Can Commit-Analyzer handle 2 different commit structure?
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
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- release
Línea de trabajo
Comienza leyendo cómo se aplican parserOpts.mergePattern y mergeCorrespondence a los mensajes de commit. Comprueba el comportamiento existente de commit-analyzer tanto para un commit fix: simple como para un commit de Azure DevOps Merged PR #####: fix:. Se considera terminado cuando ambas estructuras se reconocen y activan el análisis de versiones esperado sin requerir una configuración mutuamente excluyente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Let's say I have a commit which look like that:
fix: this is an example
Commit-analyzer is detecting this commit and triggers a patch version - which is the expected behaviour.
Then I complete a PR which creates a new commit:
Merged PR 11111: fix: this is another example
Commit-analyzer can't detect the commit fix: this is another example because of the prefix that Azure Devops adds to the commit it generates when completing a PR (Merged PR 1111: ). So I have found the mergePattern property inside parserOpts and I added a regex that inform commit-analyzer about the prefix:
parserOpts: {
mergePattern: /^Merged PR (\d+): (\w*)(?:\(([\w\$\.\-\* ]*)\))?\: (.*)((.|\n)*)/,
mergeCorrespondence: ['id', 'type', 'scope', 'subject'],
noteKeywords: ["BREAKING CHANGE", "BREAKING CHANGES"]
}
Now commit-analyzer has been able to detect the relevant commit content within this commit structure.
My problem is that now, with the mergePattern attribute defined to look for specific structure - a regular commit such as the first example fix: this is an example won't be considered as a valid commit and no version will be triggered.
There is an option to support both types of commits? with and without Merged PR #####: prefix?
Thanks.
- 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
-
area/frontend good first issue kind/cooldown
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kestra-io/kestra#20702 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Form-data fields are not parsed when importing a cURL requestPosiblemente ocupada @aayushbaluni la tomó hoy. Abiertobug status-new
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
Los mantenedores suelen responder en 1 día
-
bug MVP
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
saayam-for-all/webapp#1911 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
vshulcz/deja-vu#4937 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
[Focus] Accessibility (a11y) [Package] Grid [Type] Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día