Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

merge inhouse and non inhouse pipelines

Abierto
  • ¿Algo no está bien?
#22 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
38/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
github-actions
Área
ci-cd, infrastructure

Línea de trabajo

Compara .github/workflows/create-kustomize-with-bratiska-cli-inhouse.yml con .github/workflows/create-kustomize-with-bratiska-cli.yml para identificar la lógica de pipeline duplicada y la configuración actual del runner. Consolida los workflows en un solo archivo con un mecanismo claro de selección del runner y, después, verifica que las rutas de ejecución de GitHub y de in-house sigan disponibles y que se haya eliminado el workflow redundante.

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

Descripción

enhancement

'm starting to think that splitting this into two files wasn't the best idea. We are just copying the same code twice. Which goes directly against DRY principle.

My suggestion would be to keep just one file - .github/workflows/create-kustomize-with-bratiska-cli-inhouse.yml and drop "-inhouse" from the name. Then if you would like to run it on GitHub runners, you can just specify

runs-on: ['ubuntu-latest']
and it does the same thing and it is almost as many characters as specify "-bratiska-cli.yml" or "-bratiska-cli-inhouse.yml". Or, if you would like to be even more user friendly, you can just rework this into:

  runs-on:
    description: "Define environment where the job will run. Possible values: ['github', 'inhouse']"
    type: choice
    options: 
      - github
      - inhouse
    required: false
    default: 'inhouse'

Then you can just have a basic step that makes that decision inside a pipeline

  - name: Pick a runner
    run: |
      if [[ "${inputs.runs-on}" == "github" ]]; then
          # print and set env variable
      elif [[ "${inputs.runs-on}" == "inhouse" ]]; then
          # print and set env variable
      else
          # throw an error about unknown runner environment
      fi

This leaves the door open for multiple environments in the future and is still as user friendly as possible and you can easily switch just by specifying different input.

I'm flagging this here, as .github/workflows/create-kustomize-with-bratiska-cli-inhouse.yml and .github/workflows/create-kustomize-with-bratiska-cli.yml are new files and we can easily change it here. We don't have to make this change now for all of the files and all of the repositories. That can be pushed to latter PRs/changes.

Lenguaje dominante
Sin datos de lenguaje
Estrellas
0
Forks
0
Merge medio
3 d 3 h
PR fusionados (30 d)
1

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 bratislava/github-actions

Todos los issues de bratislava/github-actions

Issues similares

Más issues de DevOps

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.