Request: Put "name:" first in the build matrices?

Abierto
  • ¿Algo no está bien?
#110 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
35/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
github-actions, yaml
Área
ci-cd

Línea de trabajo

Start by reviewing the GitHub Actions configuration examples in this repository and compare the matrix entry ordering shown in the issue. The proposed change is to put each entry’s name field first and keep that order in the examples. The thread has no recorded follow-up since the issue was opened, so confirm the proposal still has agreement before making changes.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement

I have been editing the GHA configurations for Base and its submodules for quite some time to move the name field first in the build matrices (on the same line as the leading "-" for the block). I find the result makes it much easier to immediately see which build is being specified in each block as the name coming first summarizes it, and this also allows any of the lines setting other parameters to be copy/pasted to other blocks without having to worry about adding/removing the "-" character.

Compare:

          - os: ubuntu-22.04
            cmp: gcc
            configuration: default
            base: "3.15"
            wine: "64"
            name: "3.15 Ub-22 gcc + MinGW"

          - os: ubuntu-22.04
            cmp: gcc
            configuration: static
            base: "7.0"
            extra: "CMD_CXXFLAGS=-std=c++11"
            name: "7.0 Ub-22 gcc C++11, static"

with

          - name: "7.0 Ub gcc C++11, static"
            base: "7.0"
            os: ubuntu-latest
            cmp: gcc
            configuration: static
            extra: "CMD_CXXFLAGS=-std=c++11"

          - name: "7.0 Ub gcc u-char"
            base: "7.0"
            os: ubuntu-latest
            cmp: gcc
            configuration: static
            extra: "CMD_CFLAGS=-funsigned-char CMD_CXXFLAGS=-funsigned-char"

I'm willing to create a PR with those changes to all the examples here if there is agreement that this is a good idea, and that we should try to keep this order.

Lenguaje dominante
Python
Estrellas
8
Forks
21
Merge medio
13 h 50 min
PR fusionados (30 d)
3

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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de epics-base/ci-scripts

Todos los issues de epics-base/ci-scripts

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.